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.

classification
Title: Add 'Z' to skipitem() in Python/getargs.c
Type: behavior Stage: resolved
Components: Interpreter Core Versions: Python 3.2
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: larry Nosy List: benjamin.peterson, georg.brandl, larry, python-dev
Priority: low Keywords: patch

Created on 2012-05-08 09:56 by larry, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
larry.skipitem.Z.1.diff larry, 2012-05-08 09:56 Patch adding support for 'Z' to skipitem, for 3.2 branch.
Messages (5)
msg160191 - (view) Author: Larry Hastings (larry) * (Python committer) Date: 2012-05-08 09:56
skipitem() (in Python/getargs.c) has to be taught about all the "format units" understood by PyArg_Parse.  There's a note at the top of the format-unit-understanding code saying

  When you add new format codes,
  please don't forget poor skipitem() below.

Well, someone forgot poor skipitem() when they added 'Z'.

Since this is a bugfix, I assert it should go into 3.2, then get forward-ported into trunk.  So, step 1: check the attached one-line patch to 3.2.

Georg: sound good?
msg160193 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2012-05-08 10:22
Sound good.
msg160195 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2012-05-08 10:51
New changeset 91612618985b by Larry Hastings in branch '3.2':
Issue #14749: Add support for 'Z' to skipitem() in Python/getargs.c.
http://hg.python.org/cpython/rev/91612618985b
msg160196 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2012-05-08 10:54
New changeset b32baa5b7626 by Larry Hastings in branch 'default':
Merge from 3.2.  Issue #14749: Add support for 'Z' to skipitem().
http://hg.python.org/cpython/rev/b32baa5b7626
msg160199 - (view) Author: Benjamin Peterson (benjamin.peterson) * (Python committer) Date: 2012-05-08 12:06
No test?
History
Date User Action Args
2022-04-11 14:57:30adminsetgithub: 58954
2012-05-08 12:06:26benjamin.petersonsetnosy: + benjamin.peterson
messages: + msg160199
2012-05-08 11:20:40larrysetstatus: open -> closed
resolution: fixed
stage: patch review -> resolved
2012-05-08 10:54:20python-devsetmessages: + msg160196
2012-05-08 10:51:41python-devsetnosy: + python-dev
messages: + msg160195
2012-05-08 10:22:34georg.brandlsetmessages: + msg160193
2012-05-08 09:56:32larrycreate