[issue11550] Fix ResourceWarning in test_pulldom

2011-03-15 Thread Brett Cannon
Brett Cannon added the comment: Fixed in [aa000dd4287f]. I went with a simpler solution than Ben's, but it did help figure out what to do. -- nosy: +brett.cannon resolution: -> fixed stage: -> committed/rejected status: open -> closed ___ Python t

[issue11550] Fix ResourceWarning in test_pulldom

2011-03-15 Thread Roundup Robot
Roundup Robot added the comment: New changeset aa000dd4287f by Brett Cannon in branch 'default': Close a stream properly in test.test_pulldom. http://hg.python.org/cpython/rev/aa000dd4287f -- nosy: +python-dev ___ Python tracker

[issue11550] Fix ResourceWarning in test_pulldom

2011-03-15 Thread Nadeem Vawda
Changes by Nadeem Vawda : -- nosy: +nvawda ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.o

[issue11550] Fix ResourceWarning in test_pulldom

2011-03-15 Thread Ben Hayden
Ben Hayden added the comment: I was unclear as to why the list() calls were there. I thought it would just change the iterator into a list - I didn't know that actually was needed for the test - my mistake. I'll change it back to the list() - but I'll need to loop over the first list to make

[issue11550] Fix ResourceWarning in test_pulldom

2011-03-15 Thread Andreas Stührk
Andreas Stührk added the comment: Any reason why the patch removes the `list()` calls? Without them, no parsing happens at all. -- nosy: +Trundle ___ Python tracker ___ ___

[issue11550] Fix ResourceWarning in test_pulldom

2011-03-14 Thread Ben Hayden
Changes by Ben Hayden : -- versions: -Python 3.3, Python 3.4 Added file: http://bugs.python.org/file21205/test_pulldom_resource_warning.patch ___ Python tracker ___

[issue11550] Fix ResourceWarning in test_pulldom

2011-03-14 Thread Ben Hayden
Changes by Ben Hayden : Removed file: http://bugs.python.org/file21203/test_pulldom_resource_warning.patch ___ Python tracker ___ ___ Python-

[issue11550] Fix ResourceWarning in test_pulldom

2011-03-14 Thread Ben Hayden
New submission from Ben Hayden : When running the test_pulldom test with a latest checkout of cpython from hg.python.org on Ubuntu 10.10 x64, the following ResourceWarning is thrown: test_parse (test.test_pulldom.PullDOMTestCase) Minimal test of DOMEventStream.parse() ... /home/benhayden/Docum