[ragel-users] code review

[Nate]

Hi Adrian,

Thanks for taking a look. Yeah, I'm happy with it, I just wasn't sure I was
using it right. I'm a little worried about edge cases, where things might
parse unexpectedly. I generated a graph, but couldn't really make much use
of it.

Ragel is a really neat tool, and the documentation is very good. I think
state charts could use a little more explanation.

I wrote a little XML parser using Ragel last night, blogged about it here:
http://www.badlogicgames.com/wordpress/?p=1712

-Nate


On Mon, Mar 28, 2011 at 3:15 PM, Adrian Thurston <thurston at complang.org>wrote:

> 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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.colm.net/pipermail/ragel-users/attachments/20110330/70fa5d3d/attachment-0001.html>
-------------- next part --------------
_______________________________________________
ragel-users mailing list
ragel-users at complang.org
http://www.complang.org/mailman/listinfo/ragel-users