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.
Created on 2014年05月16日 04:52 by cvrebert, last changed 2022年04月11日 14:58 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| json-rfc-7159_1.patch | cvrebert, 2014年07月13日 21:12 | v1 of patch that updates the json module docs | review | |
| Messages (13) | |||
|---|---|---|---|
| msg218644 - (view) | Author: Chris Rebert (cvrebert) * | Date: 2014年05月16日 04:52 | |
json module docs: https://docs.python.org/3/library/json.html New superseding JSON RFC: https://tools.ietf.org/html/rfc7159 Errata to the new RFC: http://www.rfc-editor.org/errata_search.php?rfc=7159 ECMA-404: http://www.ecma-international.org/publications/standards/Ecma-404.htm These updated specs are of particular relevance to the Standard Compliance section of the json module docs. There are also a bunch of new interoperability notes in the RFC that could probably bear repeating in the module's docs. |
|||
| msg222972 - (view) | Author: Chris Rebert (cvrebert) * | Date: 2014年07月13日 21:12 | |
Here's a draft patch against the default branch that updates the json module's docs accordingly. Note that under "Implementation Limitations", the statement "This module does not impose any such limits beyond those of the relevant Python datatypes themselves or the Python interpreter itself." is just a guess; I need someone who's familiar with the implementation to verify/correct this. |
|||
| msg223038 - (view) | Author: Bob Ippolito (bob.ippolito) * (Python committer) | Date: 2014年07月14日 15:58 | |
This patch looks reasonable to me as-is. With regard to "Infinite and NaN number values are accepted and output;" there's an option for that (allow_nan=False in encoding, parse_constant=some_function_that_raises in decoding). Since an exception can't be raised in a lambda there's no simple one-liner to ensure compliant decode. |
|||
| msg223040 - (view) | Author: Chris Rebert (cvrebert) * | Date: 2014年07月14日 16:14 | |
Thanks for the speedy review! Those NaN-related arguments are already mentioned in the docs (see last 2 sentences of https://docs.python.org/3/library/json.html#infinite-and-nan-number-values ), and this patch doesn't touch that subsection. |
|||
| msg223041 - (view) | Author: Bob Ippolito (bob.ippolito) * (Python committer) | Date: 2014年07月14日 16:22 | |
Good call, I was just doing a quick review of the patch in isolation. Now I don't have anything at all to comment on :) |
|||
| msg223051 - (view) | Author: Raymond Hettinger (rhettinger) * (Python committer) | Date: 2014年07月14日 19:48 | |
Thanks Bob. If you want me to apply the patch, just assign this back to me. |
|||
| msg225136 - (view) | Author: Chris Rebert (cvrebert) * | Date: 2014年08月10日 10:30 | |
So, when might I expect to see this patch merged, since it's now been approved? |
|||
| msg225178 - (view) | Author: Raymond Hettinger (rhettinger) * (Python committer) | Date: 2014年08月11日 06:51 | |
Soonish. |
|||
| msg231055 - (view) | Author: Chris Rebert (cvrebert) * | Date: 2014年11月11日 23:51 | |
Ping! It's been about 3 months since this was given the green light... |
|||
| msg231751 - (view) | Author: Raymond Hettinger (rhettinger) * (Python committer) | Date: 2014年11月27日 06:10 | |
Serhiy, can you take this one. The patch passes both mine and Bob's review; however, it doesn't apply cleanly to 2.7 and 3.4. I haven't had the time to do the needed backports. |
|||
| msg231770 - (view) | Author: Roundup Robot (python-dev) (Python triager) | Date: 2014年11月27日 17:51 | |
New changeset 7e534e18a99a by Serhiy Storchaka in branch '2.7': Issue #21514: The documentation of the json module now refers to new JSON RFC https://hg.python.org/cpython/rev/7e534e18a99a New changeset 89bb4384f1e1 by Serhiy Storchaka in branch '3.4': Issue #21514: The documentation of the json module now refers to new JSON RFC https://hg.python.org/cpython/rev/89bb4384f1e1 New changeset aced2548345a by Serhiy Storchaka in branch 'default': Issue #21514: The documentation of the json module now refers to new JSON RFC https://hg.python.org/cpython/rev/aced2548345a |
|||
| msg231771 - (view) | Author: Chris Rebert (cvrebert) * | Date: 2014年11月27日 18:11 | |
Thanks Serhiy! |
|||
| msg231774 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) | Date: 2014年11月27日 19:31 | |
Thank you Chris for your patch. |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022年04月11日 14:58:03 | admin | set | github: 65713 |
| 2014年11月27日 19:31:05 | serhiy.storchaka | set | messages: + msg231774 |
| 2014年11月27日 19:29:04 | berker.peksag | set | status: open -> closed resolution: fixed stage: commit review -> resolved |
| 2014年11月27日 18:11:06 | cvrebert | set | messages: + msg231771 |
| 2014年11月27日 17:51:16 | python-dev | set | nosy:
+ python-dev messages: + msg231770 |
| 2014年11月27日 06:10:31 | rhettinger | set | assignee: rhettinger -> serhiy.storchaka messages: + msg231751 nosy: + serhiy.storchaka |
| 2014年11月12日 01:17:25 | berker.peksag | set | nosy:
+ berker.peksag |
| 2014年11月11日 23:51:08 | cvrebert | set | messages: + msg231055 |
| 2014年08月11日 09:24:58 | berker.peksag | set | stage: needs patch -> commit review |
| 2014年08月11日 06:51:49 | rhettinger | set | messages: + msg225178 |
| 2014年08月10日 10:30:29 | cvrebert | set | messages: + msg225136 |
| 2014年07月14日 19:53:23 | bob.ippolito | set | assignee: bob.ippolito -> rhettinger |
| 2014年07月14日 19:48:10 | rhettinger | set | messages: + msg223051 |
| 2014年07月14日 16:22:54 | bob.ippolito | set | messages: + msg223041 |
| 2014年07月14日 16:14:59 | cvrebert | set | messages: + msg223040 |
| 2014年07月14日 15:58:43 | bob.ippolito | set | messages: + msg223038 |
| 2014年07月14日 08:56:48 | rhettinger | set | assignee: docs@python -> bob.ippolito nosy: + bob.ippolito |
| 2014年07月13日 21:12:52 | cvrebert | set | files:
+ json-rfc-7159_1.patch keywords: + patch messages: + msg222972 |
| 2014年05月16日 07:15:07 | ezio.melotti | set | nosy:
+ rhettinger, pitrou, ezio.melotti stage: needs patch type: enhancement versions: + Python 3.4 |
| 2014年05月16日 04:52:48 | cvrebert | create | |