[racket-dev] request for code review on fix for pr 13471
It looks ok to me.
At Sun, 3 Feb 2013 22:26:00 -0700, Danny Yoo wrote:
> I've run the tests that use the lexer, and it appears to pass. But
> since it's parser-tools/lex, I'd like a second opinion on this, just
> in case I'm doing something very silly.
>
> I've put this in my branch 'pr13471':
>
> https://github.com/dyoo/racket/tree/pr13471
>
> After someone takes a look and says ok, I can push the fix upstream.
>
>
> On Fri, Feb 1, 2013 at 2:44 PM, Danny Yoo <dyoo at hashcollision.org> wrote:
> > I've written up a patch for PR 13471:
> >
> >
> https://github.com/dyoo/racket/commit/544918465b67cbf2e8ad40bd3c2b3ea90f33c085
> >
> > I just want a second pair of eyes just to make sure I've done it
> > correctly. I added a free-id-table that the parse function uses when
> > walking across lex-abbrev syntaxes. Does this look ok?
> _________________________
> Racket Developers list:
> http://lists.racket-lang.org/dev