Terry J. Reedy added the comment:
Nasty bug. Running a file with 'coding=0', a quite legitimate assignment
statement, causes Idle to close, with LookupError, leading to SyntaxError,
reported on the console if there is one ('crash' otherwise). (Idle closing is a
separate problem, with an issue, from the misinterpretation of 'coding'.)
Loading such a file works with a warning that should not be there.
Adding # leads to "SyntaxError: unknown encoding" in a message box, without
closing Idle. I presume this is to be expected and is proper. There is also a
warning on loading.
The code patch adds '^[ \t\f]' to the re. \f = FormFeed? Should that really be
there? The PEP patch instead adds '^[ \t\v]', \v= VerticalTab? Same question,
and why the difference?
Your other changes to IOBinding.coding_spec look correct and fix a couple of
bugs in the function (searching all lines for the coding cookie, mangling a
line without a line end).
Someone else should review the other code changes.
----------
_______________________________________
Python tracker <[email protected]>
<http://bugs.python.org/issue18873>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
