Terry J. Reedy added the comment:
Summary of this post: compile currently checks user input with assert; this is
a bug that should be changed.
I re-read astlinenotest.py and realized that FunctionDef is included in '*' and
not some omitted import. (The latter is common for code posted on python-list,
if not here. This illustrates why PEP 8 deprecates 'import *'; import ast ...
ast.FunctionDef would have been clear on first reading.) So I ran the module in
installed 3.4 and repository debug 3.5 and got the assert with the latter.
"It seems to me that when the lnotab was created, no one even had in mind that
there would be an actually useful AST module that would be used for code
generation." I am pretty sure this is correct. I suspect that the assert in
question was originally intended to test the logic of the internal syntax tree
line number generation. But now it also tests user input, which I and may
others think is a bad idea.
A solution between removing the assert (and the internal check) and converting
it to, say, ValueError("decreasing line no in input ast"), and thereby stopping
code that now normally and legitimately works, would be to skip the asserts
when the compile input is an ast instead of code.
----------
stage: -> needs patch
_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue21385>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
