This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author ezio.melotti
Recipients cvrebert, docs@python, eric.araujo, ezio.melotti, serhiy.storchaka
Date 2012-05-15.08:26:30
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1337070390.92.0.205403793137.issue14674@psf.upfronthosting.co.za>
In-reply-to
Content
Thanks for the patch, I left more comments on the review page.

IMHO it would be better to list the differences in a bullet list and expand later, rather than having a section for the parser and one for the generator.
AFAIU the differences are:
 * top-level scalar values are accepted/generated;
 * inf and nan are accepted/generated;
 * unicode strings (rather than utf-8) are produced/consumed;
 * duplicate keys are accepted, and the only the last one is used;

You can then add examples and explain "workarounds", either inline or after the list (I don't think it's necessary to add the snippets you posted in the last message though).


> Sidenote: Is there any way to get notified of these reviews?

In theory you should get notifications, in practice it doesn't always work.  We are still working on make it better.
History
Date User Action Args
2012-05-15 08:26:30ezio.melottisetrecipients: + ezio.melotti, eric.araujo, cvrebert, docs@python, serhiy.storchaka
2012-05-15 08:26:30ezio.melottisetmessageid: <1337070390.92.0.205403793137.issue14674@psf.upfronthosting.co.za>
2012-05-15 08:26:30ezio.melottilinkissue14674 messages
2012-05-15 08:26:30ezio.melotticreate