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.

Author JohnLeitch
Recipients BreamoreBoy, JohnLeitch, belopolsky, brycedarling, eryksun, georg.brandl, larry, lemburg, paul.moore, python-dev, steve.dower, tim.golden, vstinner, zach.ware
Date 2015-09-06.05:30:57
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1441517457.58.0.698560349236.issue24917@psf.upfronthosting.co.za>
In-reply-to
Content
Yikes--your comment prompted me to look at the check-in, and it seems my patch wasn't properly applied. The curly braces got tweaked, which is minor as you stated, but more importantly the AIX code should not decref format. That could introduce problems bigger than what this patch was attempting to fix.

And, not to dwell, but where do you see a keyword immediately followed by a left parens? I want to make sure everything is properly polished in the future, and the only thing I see is the untouched "for".

Regarding your initial concerns:
1) I think we should enforce no trailing % so as to not pass format strings that may cause undefined behavior.

2) How about expecting ValueError on Windows/AIX, and pass on all other platforms?
History
Date User Action Args
2015-09-06 05:30:57JohnLeitchsetrecipients: + JohnLeitch, lemburg, georg.brandl, paul.moore, belopolsky, vstinner, larry, tim.golden, BreamoreBoy, python-dev, zach.ware, eryksun, steve.dower, brycedarling
2015-09-06 05:30:57JohnLeitchsetmessageid: <1441517457.58.0.698560349236.issue24917@psf.upfronthosting.co.za>
2015-09-06 05:30:57JohnLeitchlinkissue24917 messages
2015-09-06 05:30:57JohnLeitchcreate