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 <[email protected]> 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 _________________________ Racket Developers list: http://lists.racket-lang.org/dev
