]> granicus.if.org Git - python/commit
Upon insertion, if memory runs out, the deque was left in a corrupted state.
authorArmin Rigo <arigo@tunes.org>
Sat, 2 Oct 2004 13:59:34 +0000 (13:59 +0000)
committerArmin Rigo <arigo@tunes.org>
Sat, 2 Oct 2004 13:59:34 +0000 (13:59 +0000)
commit974d757af15c22049a2aaa8cecf1456cd79d6397
tree1bae979021d1710798b4cc979b3dcdaa1cdfbcef
parent565ea5ae3701b3600cb7ac2f6405574b51c19ba7
Upon insertion, if memory runs out, the deque was left in a corrupted state.

deque_item(): a performance bug: the linked list of blocks was followed
from the left in most cases, because the test (i < (deque->len >> 1)) was
after "i %= BLOCKLEN".

deque_clear(): replaced a call to deque_len() with deque->len; not sure what
this call was here for, nor if all compilers under the sun would inline it.

deque_traverse(): I belive that it could be called by the GC when the deque
has leftblock==rightblock==NULL, because it is tracked before the first block
is allocated (though closely before).  Still, a C extension module subclassing
deque could provide its own tp_alloc that could trigger a GC collection after
the PyObject_GC_Track()...

deque_richcompare(): rewrote to cleanly check for end-of-iterations instead of
relying on deque.__iter__().next() to succeed exactly len(deque) times -- an
assumption which can break if deques are subclassed.  Added a test.

I wonder if the length should be explicitely bounded to INT_MAX, with
OverflowErrors, as in listobject.c.  On 64-bit machines, adding more than
INT_MAX in the deque will result in trouble.  (Note to anyone/me fixing
this: carefully check for overflows if len is close to INT_MAX in the
following functions: deque_rotate(), deque_item(), deque_ass_item())
Lib/test/test_deque.py
Modules/collectionsmodule.c