[ragel-users] code review

[Adrian Thurston]

Hi Nate,

My apologies for taking so long to look at this. Code reviews are 
sometimes quite involved so I often put them off ... I just grabbed the 
latest and it seems the number of states is down to 113. That is 
certainly reasonable for a ragel program. Are you happy with it?

-Adrian

On 02/23/2011 07:03 PM, Nate wrote:
> Hi,
>
> I am using Ragel to parse a DSL for grid-based GUI layouts. What I have
> is working, but I'm not sure it is done well. I would love feedback on
> how it can be improved. I generated a graph with Graphviz... it was 10mb
> and
> 32767x5625 pixels, which makes me think maybe I'm not doing this right.
>
> My Ragel file:
> http://code.google.com/p/table-layout/source/browse/trunk/build/TableLayoutParser.rl
>
> My project, with a description of the language:
> http://code.google.com/p/table-layout/
>
> The language is hierarchical. I'm using fjump/fret to handle that. Also,
> FWIW, I chose Ragel over JavaCC and ANTLR because I like the small
> amount of code generated. I also don't like that ANTLR has a runtime
> dependency.
>
> Thanks!
> -Nate
>
>
>
> _______________________________________________
> ragel-users mailing list
> ragel-users at complang.org
> http://www.complang.org/mailman/listinfo/ragel-users
>
> --===============1214348

-- 
Adrian D. Thurston
http://www.complang.org/thurston/

_______________________________________________
ragel-users mailing list
ragel-users at complang.org
http://www.complang.org/mailman/listinfo/ragel-users