[issue16795] Patch: some changes to AST to make it more useful for static language analysis

Sven Brauch report at bugs.python.org
Tue Jan 29 23:07:27 CET 2013


Sven Brauch added the comment:

Hm, I'm still getting the same error messages from the review tool which I described earlier; I can neither comment nor add patches.

So, I'll have to abuse the bug report again:
Thanks for the review. Is it possible you selected the wrong patch file for the second patch (Patch Set 5)? It seems to include all changes instead of just those from the second of the three patches I submitted. Also I had already removed the traceback.print_stack() call.

I fixed the other two issues and I will attach a corrected version of the second patch for review. I hope I got everything right ;)

Please have an extra close look at the changes to symtable.c and compile.c (since I'm not very familiar with that code), in order to avoid that we break stuff with this.

Cheers,
Sven

----------
Added file: http://bugs.python.org/file28905/81300-change-var-kwargs-new.diff

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue16795>
_______________________________________


More information about the Python-bugs-list mailing list