[issue19018] Heapq.merge suppreses IndexError from user generator

2013-09-14 Thread Artem Fokin
Artem Fokin added the comment: Which branch should I add a unit-test to? Here is a patch that adds a unit-test to the current one. -- keywords: +patch Added file: http://bugs.python.org/file31760/unittest_patch.diff ___ Python tracker <h

[issue19018] Heapq.merge suppreses IndexError from user generator

2013-09-14 Thread Artem Fokin
Artem Fokin added the comment: Oh, it seems that in current cpython branch this problem is fixed by checking condition _len(h) > 1: http://hg.python.org/cpython/file/1dc925ee441a/Lib/heapq.py#l373 But is it possible to fix it for the previous branc

[issue19018] Heapq.merge suppreses IndexError from user generator

2013-09-14 Thread Artem Fokin
Changes by Artem Fokin : -- versions: -3rd party, Python 2.6, Python 3.4, Python 3.5 ___ Python tracker <http://bugs.python.org/issue19018> ___ ___ Python-bug

[issue19018] Heapq.merge suppreses IndexError from user generator

2013-09-14 Thread Artem Fokin
New submission from Artem Fokin: Suppose we have the following code: from heapq import merge def iterable(): lst = range(10) for i in xrange(20): yield lst[i] it1, it2= iterable(), iterable() print list(merge(it1, it2)) # no IndexError