Repository navigation
Document effects of PY_SSIZE_T_CLEAN on argument parsing #45023
Description
| BPO | 1729742 |
|---|---|
| Nosy | @loewis, @birkenfeld, @ashemedai |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
assignee = 'https://github.com/birkenfeld'
closed_at = <Date 2009-04-25.11:39:46.419>
created_at = <Date 2007-06-01.23:51:25.000>
labels = ['docs']
title = 'Document effects of PY_SSIZE_T_CLEAN on argument parsing'
updated_at = <Date 2009-04-25.11:39:46.418>
user = 'https://bugs.python.org/bwelling'bugs.python.org fields:
activity = <Date 2009-04-25.11:39:46.418>
actor = 'loewis'
assignee = 'georg.brandl'
closed = True
closed_date = <Date 2009-04-25.11:39:46.419>
closer = 'loewis'
components = ['Documentation']
creation = <Date 2007-06-01.23:51:25.000>
creator = 'bwelling'
dependencies = []
files = []
hgrepos = []
issue_num = 1729742
keywords = []
message_count = 4.0
messages = ['32175', '32176', '86449', '86477']
nosy_count = 4.0
nosy_names = ['loewis', 'georg.brandl', 'bwelling', 'asmodai']
pr_nums = []
priority = 'high'
resolution = 'fixed'
stage = None
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue1729742'
versions = ['Python 2.5']
Activity
bwelling commentedon Jun 1, 2007
The "Parsing arguments and building values" section of the Python/C API Reference Manual describes the "s#" format unit as requiring a [const char *, int]. This should presumably be a Py_ssize_t, as that's what the code appears to expect.
The same problem also occurs elsewhere in this page; pretty much everywhere the # format modifier is described.
I'm seeing this in both the 2.5.1 and development versions of the docs on docs.python.org.
loewis commentedon Jun 2, 2007
It's more complicated than that, see PEP-353: It's Py_ssize_t if the module defines PY_SSIZE_T_CLEAN, else it's int. Would you like to work on a patch fixing the documentation?
[-]missing int->Py_ssize_t in documentation[/-][+]Document effects of PY_SSIZE_T_CLEAN on argument parsing[/+][-]missing int->Py_ssize_t in documentation[/-][+]Document effects of PY_SSIZE_T_CLEAN on argument parsing[/+]ashemedai commentedon Apr 25, 2009
This seems to be present in c-api/arg.rst in trunk already.
Candidate for closure after verifying it's been merged appropriately?
loewis commentedon Apr 25, 2009
Closing as suggested