[issue16089] _elementtree.TreeBuilder broken with a non-C-deriving element_factory

2012-11-09 Thread Dmitry Shachnev
Dmitry Shachnev added the comment: There are still some false-positive warnings caused by C code that affect docutils (see http://sourceforge.net/tracker/?func=detailatid=422030aid=3555164group_id=38414). Look at this traceback for exmaple: http://paste.ubuntu.com/1345164/ There,

[issue16089] _elementtree.TreeBuilder broken with a non-C-deriving element_factory

2012-11-09 Thread Dmitry Shachnev
Changes by Dmitry Shachnev mity...@gmail.com: ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16089 ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue16089] _elementtree.TreeBuilder broken with a non-C-deriving element_factory

2012-11-09 Thread Dmitry Shachnev
Changes by Dmitry Shachnev mity...@gmail.com: ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16089 ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue16089] _elementtree.TreeBuilder broken with a non-C-deriving element_factory

2012-10-04 Thread Roundup Robot
Roundup Robot added the comment: New changeset 7bd9626d8b4f by Antoine Pitrou in branch '3.3': Issue #16089: Allow ElementTree.TreeBuilder to work again with a non-Element element_factory (fixes a regression in SimpleTAL). http://hg.python.org/cpython/rev/7bd9626d8b4f New changeset

[issue16089] _elementtree.TreeBuilder broken with a non-C-deriving element_factory

2012-10-04 Thread Antoine Pitrou
Changes by Antoine Pitrou pit...@free.fr: -- resolution: - fixed stage: patch review - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16089 ___

[issue16089] _elementtree.TreeBuilder broken with a non-C-deriving element_factory

2012-10-04 Thread Eli Bendersky
Eli Bendersky added the comment: Antoine, Thanks! You said : the coding style there is quite old It would be great if you could elaborate, however briefly, if there's anything else besides your fixes that is old and should be modernized. I will admit to writing some of that code very

[issue16089] _elementtree.TreeBuilder broken with a non-C-deriving element_factory

2012-10-04 Thread Antoine Pitrou
Antoine Pitrou added the comment: the coding style there is quite old It would be great if you could elaborate, however briefly, if there's anything else besides your fixes that is old and should be modernized. I will admit to writing some of that code very recently, but that's mostly

[issue16089] _elementtree.TreeBuilder broken with a non-C-deriving element_factory

2012-10-04 Thread Eli Bendersky
Eli Bendersky added the comment: Ah, OK. I was actually putting off refactorings in that code to after refactoring the test suite. The latter is in progress, it was paused by my vacation but I do plan to resume it eventually. Once the test suite is sufficiently humane (esp. moves off doctest

[issue16089] _elementtree.TreeBuilder broken with a non-C-deriving element_factory

2012-10-02 Thread Antoine Pitrou
Antoine Pitrou added the comment: Here is a patch. It still needs some tests. -- title: _elementtree causes segfault in GC - _elementtree.TreeBuilder broken with a non-C-deriving element_factory Added file: http://bugs.python.org/file27392/c_treebuilder.patch

[issue16089] _elementtree.TreeBuilder broken with a non-C-deriving element_factory

2012-10-02 Thread Antoine Pitrou
Antoine Pitrou added the comment: Updated patch with tests. -- stage: needs patch - patch review Added file: http://bugs.python.org/file27393/c_treebuilder2.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16089