This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author rhettinger
Recipients rhettinger
Date 2015-09-27.08:43:40
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1443343421.14.0.18683145075.issue25246@psf.upfronthosting.co.za>
In-reply-to
Content
The current algorithm for remove() rotates the deque one-by-one until a match is found, pops it off the deque and does single mass rotate to undo the 1-step rotates.

An alternative approach is to use deque_index() to locate the value of interest and use deque_del_item() to remove it.  

If not value is found, the alternative is better because it never moves the data in the deque.  If the value is found, the alternative removes it using two mass rotations.  The advantage in that case is the mass rotates are faster than many 1-step rotates.  The disadvantage is that we go through the pointer chain twice (the first time visiting and comparing every element and the second time only following the chain of links).

If the deque mutates during the search, a RuntimeError is raised.  This is a behavior change, formerly it raised an IndexError.
History
Date User Action Args
2015-09-27 08:43:41rhettingersetrecipients: + rhettinger
2015-09-27 08:43:41rhettingersetmessageid: <1443343421.14.0.18683145075.issue25246@psf.upfronthosting.co.za>
2015-09-27 08:43:41rhettingerlinkissue25246 messages
2015-09-27 08:43:40rhettingercreate