Issue 24007: Write PyArg_Parse* format in a line with a function (original) (raw)

Issue24007

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.

This issue has been migrated to GitHub: https://github.com/python/cpython/issues/68195

classification

Title: Write PyArg_Parse* format in a line with a function
Type: enhancement Stage: resolved
Components: Argument Clinic, Demos and Tools Versions: Python 3.5

process

Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: serhiy.storchaka Nosy List: larry, python-dev, serhiy.storchaka
Priority: normal Keywords: patch

Created on 2015-04-19 18:30 by serhiy.storchaka, last changed 2022-04-11 14:58 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
clinic_format_argument.patch serhiy.storchaka,2015-04-19 18:30 review
Messages (4)
msg241531 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2015-04-19 18:30
Proposed patch makes Argument Clinic to output format argument of PyArg_Parse, PyArg_ParseTuple and PyArg_ParseTupleAndKeywords in a line with a function itself. First, this makes generated code more compact and easier to read and compare with old code. Second, it makes easier to grep sources for used format units. Format argument always (or almost always) is written in a line with a function.
msg241567 - (view) Author: Larry Hastings (larry) * (Python committer) Date: 2015-04-19 21:59
I don't care about this, but the patch looks fine. If this really helps then LGTM. Please hold off checking this in until after 3.5.0 alpha 4 is released.
msg241607 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2015-04-20 05:36
Actually it is not hard to me to apply the patch locally and regenerate clinic output if I need it. I propose this patch for mainstream only because think that it can be helpful for someone other. Thanks to moving all generated code into separated files, the code churn does not harm.
msg241893 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2015-04-23 21:41
New changeset 151cab576cab by Serhiy Storchaka in branch 'default': Issue #24007: Argument Clinic now writes the format of PyArg_Parse*() at the https://hg.python.org/cpython/rev/151cab576cab
History
Date User Action Args
2022-04-11 14:58:15 admin set github: 68195
2015-04-23 21:41:55 serhiy.storchaka set status: open -> closedassignee: serhiy.storchakaresolution: fixedstage: patch review -> resolved
2015-04-23 21:41:19 python-dev set nosy: + python-devmessages: +
2015-04-20 05:36:03 serhiy.storchaka set messages: +
2015-04-19 21:59:38 larry set messages: +
2015-04-19 18:30:47 serhiy.storchaka create