From: Raymond Hettinger Date: Sat, 6 Jul 2013 19:07:06 +0000 (-1000) Subject: Remove unnecessary branches from count() and reverse(). X-Git-Tag: v3.4.0a1~319 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=98054b4c1be933f5bbfb3c1a1f4ee3f556dd47ca;p=python Remove unnecessary branches from count() and reverse(). --- diff --git a/Modules/_collectionsmodule.c b/Modules/_collectionsmodule.c index 781949d48f..376a8e6770 100644 --- a/Modules/_collectionsmodule.c +++ b/Modules/_collectionsmodule.c @@ -551,6 +551,8 @@ deque_reverse(dequeobject *deque, PyObject *unused) for (i=0 ; idata[leftindex]; @@ -560,8 +562,6 @@ deque_reverse(dequeobject *deque, PyObject *unused) /* Advance left block/index pair */ leftindex++; if (leftindex == BLOCKLEN) { - if (leftblock->rightlink == NULL) - break; leftblock = leftblock->rightlink; leftindex = 0; } @@ -569,8 +569,6 @@ deque_reverse(dequeobject *deque, PyObject *unused) /* Step backwards with the right block/index pair */ rightindex--; if (rightindex == -1) { - if (rightblock->leftlink == NULL) - break; rightblock = rightblock->leftlink; rightindex = BLOCKLEN - 1; } @@ -594,6 +592,7 @@ deque_count(dequeobject *deque, PyObject *v) int cmp; for (i=0 ; idata[leftindex]; cmp = PyObject_RichCompareBool(item, v, Py_EQ); if (cmp > 0) @@ -610,8 +609,6 @@ deque_count(dequeobject *deque, PyObject *v) /* Advance left block/index pair */ leftindex++; if (leftindex == BLOCKLEN) { - if (leftblock->rightlink == NULL) /* can occur when i==n-1 */ - break; leftblock = leftblock->rightlink; leftindex = 0; }