From: Raymond Hettinger Date: Sun, 21 Jul 2013 08:51:07 +0000 (-0700) Subject: Minor code simplification by eliminating an unnecessary temporary variable. X-Git-Tag: v3.4.0a1~120 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=b97cc49c3a4870713f1872de87543651dc940e96;p=python Minor code simplification by eliminating an unnecessary temporary variable. --- diff --git a/Modules/_collectionsmodule.c b/Modules/_collectionsmodule.c index 5d5fc912a1..3ff1d1fc7e 100644 --- a/Modules/_collectionsmodule.c +++ b/Modules/_collectionsmodule.c @@ -491,7 +491,6 @@ _deque_rotate(dequeobject *deque, Py_ssize_t n) b = NULL; } assert(leftindex > 0); - { PyObject **src, **dest; Py_ssize_t m = n; @@ -510,15 +509,13 @@ _deque_rotate(dequeobject *deque, Py_ssize_t n) *(dest--) = *(src--); } while (--m); } - if (rightindex == -1) { - block *prevblock = rightblock->leftlink; assert(leftblock != rightblock); assert(b == NULL); b = rightblock; - CHECK_NOT_END(prevblock); - MARK_END(prevblock->rightlink); - rightblock = prevblock; + CHECK_NOT_END(rightblock->leftlink); + rightblock = rightblock->leftlink; + MARK_END(rightblock->rightlink); rightindex = BLOCKLEN - 1; } } @@ -538,7 +535,6 @@ _deque_rotate(dequeobject *deque, Py_ssize_t n) b = NULL; } assert (rightindex < BLOCKLEN - 1); - { PyObject **src, **dest; Py_ssize_t m = -n; @@ -557,15 +553,13 @@ _deque_rotate(dequeobject *deque, Py_ssize_t n) *(dest++) = *(src++); } while (--m); } - if (leftindex == BLOCKLEN) { - block *nextblock = leftblock->rightlink; assert(leftblock != rightblock); assert(b == NULL); b = leftblock; - CHECK_NOT_END(nextblock); - MARK_END(nextblock->leftlink); - leftblock = nextblock; + CHECK_NOT_END(leftblock->rightlink); + leftblock = leftblock->rightlink; + MARK_END(leftblock->leftlink); leftindex = 0; } }