[issue14674] Add link to RFC 4627 from json documentation
Éric Araujo mer...@netwok.org added the comment: A message should have been sent to pyb...@rebertia.com. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Chris Rebert pyb...@rebertia.com added the comment: So, does the refactored patch need any further revising, or is it good to go? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Changes by Chris Rebert pyb...@rebertia.com: Removed file: http://bugs.python.org/file25592/json.rst.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Changes by Chris Rebert pyb...@rebertia.com: Added file: http://bugs.python.org/file25606/json.rst.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Changes by Chris Rebert pyb...@rebertia.com: Removed file: http://bugs.python.org/file25591/json.rst.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Chris Rebert pyb...@rebertia.com added the comment: New revision per Éric's Rietveld feedback. Sidenote: Is there any way to get notified of these reviews? I only saw it because I happened to click the review link on a lark. -- Added file: http://bugs.python.org/file25594/json.rst.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Chris Rebert pyb...@rebertia.com added the comment: The import jsons were left for uniformity with the other code samples in the module's docs. Also, here's what the pedantically-strict recipes might look like: def _reject_inf_nan(string): if string in {'-Infinity', 'Infinity', 'NaN'}: raise ValueError(JSON does not allow infinite or NaN number values) def _reject_dupe_keys(pairs): obj = {} for key, value in pairs: if key in pairs: raise ValueError(Name %s repeated in an object % repr(key)) obj[key] = value return obj def strict_loads(string): result = loads(string, parse_constant=_reject_inf_nan, object_pairs_hook=_reject_dupe_keys) if not isinstance(result, (dict, list)): raise ValueError(The top-level entity of the JSON text was not an object or an array) return result def strict_dumps(obj): if not isinstance(obj, (dict, list)): raise TypeError(The top-level object of a JSON text must be a dict or a list) return dumps(obj, allow_nan=False) -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Ezio Melotti ezio.melo...@gmail.com added the comment: 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. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Chris Rebert pyb...@rebertia.com added the comment: Further copyediting. -- Added file: http://bugs.python.org/file25595/json.rst.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Serhiy Storchaka storch...@gmail.com added the comment: for key, value in pairs: if key in pairs: if key in obj:? -- title: Link to explain deviations from RFC 4627 in json module docs - Add link to RFC 4627 from json documentation ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Serhiy Storchaka storch...@gmail.com added the comment: IMHO, it would be sufficient to have a simple bullet list of differences and notes or warnings in places where Python can generate non-standard JSON (top-level scalars, inf and nan, non-utf8 encoded strings). -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Chris Rebert pyb...@rebertia.com added the comment: for key, value in pairs: if key in pairs: if key in obj:? Yes, obviously. :-) It wrote those very late at night. @Ezio: These were per Éric's feedback but would be for a separate bug/patch. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Chris Rebert pyb...@rebertia.com added the comment: Notification of any future Rietveld comments would be appreciated. -- Added file: http://bugs.python.org/file25603/json.rst.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Chris Rebert pyb...@rebertia.com added the comment: Draft docs patch. More cross-referencing. Analysis of deviations from RFC based on previous docs the RFC. Don't know if there are relevant, more precise implementation limitations that need to be mentioned. If backported to 2.x, will need to cover encodings slightly more. -- keywords: +patch nosy: +cvrebert Added file: http://bugs.python.org/file25591/json.rst.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Changes by Chris Rebert pyb...@rebertia.com: Added file: http://bugs.python.org/file25592/json.rst.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Changes by Ezio Melotti ezio.melo...@gmail.com: -- nosy: +ezio.melotti versions: +Python 2.7, Python 3.2 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Nurhusien Hasen nurhusien.hase...@gmail.com added the comment: find python truste .net all serves -- nosy: +Nurhusien.Hasen ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Nurhusien Hasen nurhusien.hase...@gmail.com added the comment: find python .org public all serves On 5/6/12, Nurhusien Hasen rep...@bugs.python.org wrote: Nurhusien Hasen nurhusien.hase...@gmail.com added the comment: find python truste .net all serves -- nosy: +Nurhusien.Hasen ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Changes by Ezio Melotti ezio.melo...@gmail.com: -- Removed message: http://bugs.python.org/msg160089 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Changes by Ezio Melotti ezio.melo...@gmail.com: -- Removed message: http://bugs.python.org/msg160098 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Changes by Ezio Melotti ezio.melo...@gmail.com: -- nosy: -Nurhusien.Hasen ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
Changes by Éric Araujo mer...@netwok.org: -- keywords: +easy nosy: +eric.araujo stage: - needs patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14674] Add link to RFC 4627 from json documentation
New submission from Serhiy Storchaka storch...@gmail.com: The json module documentation should give a link to RFC 4627 and explain the current implementation is different from it. For example, according to RFC 4627 only an object or an array can be top-level JSON element. -- assignee: docs@python components: Documentation messages: 159367 nosy: docs@python, storchaka priority: normal severity: normal status: open title: Add link to RFC 4627 from json documentation type: enhancement versions: Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14674 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com