Re: [Dxfreader-info] Parser Refactor

[Allan D. <al...@br...>]

Hi Mike,

I'm finally sitting down with your changes and watching the google  
video you send on the code review process.

I'm not familiar with the diff format you provided. What tool are you  
using to do that? Can you send a link to something where I can learn a  
bit about the diff file format?

Thanks.

-Allan


On Mar 16, 2009, at 10:10 PM, Michael Caron wrote:

> Hi Allan (i.e. list),
>
> I made some good headway on Saturday and this evening with the  
> parser refactor. Please take a look and let me know what you think.  
> Personally, I think that with a parser object, the division of labor  
> is much clearer. Also, a parser then can keep track of where the  
> index is in the file and move it forward and backward, reset it, etc.
>
> The attached diff includes my current work so far and unit tests.  
> Let me know what you think. I won't submit it until the  
> functionality of the entity parsing matches your previous work.
>
> <ParserRefactor.diff>
>
> Michael
> ------------------------------------------------
> mic...@gm...
> http://techrad.wordpress.com
> http://mrcaron.wordpress.com
>
> ------------------------------------------------------------------------------
> Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM)  
> are
> powering Web 2.0 with engaging, cross-platform capabilities. Quickly  
> and
> easily build your RIAs with Flex Builder, the Eclipse(TM)based  
> development
> software that enables intelligent coding and step-through debugging.
> Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com_______________________________________________
> Dxfreader-info mailing list
> Dxf...@li...
> https://lists.sourceforge.net/lists/listinfo/dxfreader-info