Message 145651 - Python tracker

➜

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	vstinner
Recipients	Arfrever, georg.brandl, vstinner
Date	2011-10-16.23:29:45
SpamBayes Score	0.01653416
Marked as misclassified	No
Message-id	<1318807786.06.0.739508380718.issue13032@psf.upfronthosting.co.za>
In-reply-to

Content
Using your patch, h2py.py skips all statements that cannot be decoded from UTF-8, whereas unpatched h2py.py accepts all statements that can be decoded from the locale encoding. I don't know if it is intentional to accept non-ASCII statements. It is maybe safer to ensure that a statement is encodable to ASCII using: try: stmt.encode('ASCII') exec(stmt, env) except: ... Anyway, I would prefer to just drop this script with all Lib/plat-*/ directoes. I reopened this topic on python-dev (I already asked when I was working on the sys.platform=="linux3" issue: #12326).

Using your patch, h2py.py skips all statements that cannot be decoded from UTF-8, whereas unpatched h2py.py accepts all statements that can be decoded from the locale encoding. I don't know if it is intentional to accept non-ASCII statements. It is maybe safer to ensure that a statement is encodable to ASCII using:

try:
    stmt.encode('ASCII')
    exec(stmt, env)
except:
    ...

Anyway, I would prefer to just drop this script with all Lib/plat-*/ directoes. I reopened this topic on python-dev (I already asked when I was working on the sys.platform=="linux3" issue: #12326).

History
Date	User	Action	Args
2011-10-16 23:29:46	vstinner	set	recipients: + vstinner, georg.brandl, Arfrever
2011-10-16 23:29:46	vstinner	set	messageid: <1318807786.06.0.739508380718.issue13032@psf.upfronthosting.co.za>
2011-10-16 23:29:45	vstinner	link	issue13032 messages
2011-10-16 23:29:45	vstinner	create