[issue10869] ast.increment_lineno() increments root node twice

2011-01-08 Thread Georg Brandl
Georg Brandl added the comment: Thanks, fixed in r87876. Will backport to other branches. -- nosy: +georg.brandl resolution: -> accepted status: open -> closed ___ Python tracker

[issue10869] ast.increment_lineno() increments root node twice

2011-01-08 Thread Éric Araujo
Changes by Éric Araujo : -- keywords: +needs review nosy: +aronacher, rhettinger resolution: fixed -> stage: -> patch review ___ Python tracker ___

[issue10869] ast.increment_lineno() increments root node twice

2011-01-08 Thread David White
David White added the comment: I've uploaded the patches. Beware, this is my first attempt to contribute. -- ___ Python tracker ___

[issue10869] ast.increment_lineno() increments root node twice

2011-01-08 Thread David White
Changes by David White : -- resolution: -> fixed Added file: http://bugs.python.org/file20319/increment_linenos_3k.patch ___ Python tracker ___ _

[issue10869] ast.increment_lineno() increments root node twice

2011-01-08 Thread David White
Changes by David White : Added file: http://bugs.python.org/file20318/increment_linenos_31.patch ___ Python tracker ___ ___ Python-bugs-list m

[issue10869] ast.increment_lineno() increments root node twice

2011-01-08 Thread David White
Changes by David White : -- keywords: +patch Added file: http://bugs.python.org/file20317/increment_linenos_27.patch ___ Python tracker ___ __

[issue10869] ast.increment_lineno() increments root node twice

2011-01-08 Thread David White
New submission from David White : When using ast.increment_lineno(), it visits the root node twice. Therefore if the root node contains the lineno attribute, it is incremented twice. (Often, the root node will not have a lineno attribute, so there is no effect. This is why the current test