[racket-dev] [plt] Push #25403: master branch updated

On Fri, Oct 5, 2012 at 11:47 AM, Eli Barzilay <eli at barzilay.org> wrote:

> On Tuesday, Carl Eastlund wrote:
> > Oops, I hadn't meant to push that to the main repository without
> > consulting anyone.  Hope this change is okay, it seemed weird for
> > read-json to error instead of producing eof.
>
> I think that the change is ok (and the added test shows a point for
> doing it).  But the implementation is broken:
>
>   -> (string->jsexpr "]")
>   #<eof>
>   -> (string->jsexpr "ndljvnldj")
>   #<eof>
>
> Also, there's no reason for a keyword there, right?
>
> (I'm committing a fixed version.)
>

Thanks for the fix.  And no, the keyword doesn't matter.

--Carl
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.racket-lang.org/dev/archive/attachments/20121005/a716beff/attachment.html>