[Patches] [Patch #100884] arraymodule: adding count, extend, index, pop, remove

noreply@sourceforge.net noreply@sourceforge.net
Mon, 31 Jul 2000 06:30:52 -0700


Patch #100884 has been updated. 

Project: 
Category: None
Status: Rejected
Summary: arraymodule: adding count, extend, index, pop, remove

Follow-Ups:

Date: 2000-Jul-31 01:45
By: nowonder

Comment:
needs reviewing. includes test and documentation update. works fine on all tests.
-------------------------------------------------------

Date: 2000-Jul-31 06:30
By: gvanrossum

Comment:
Rejected: the idea is good, but the code buggy:

- getarrayitem() returns a new reference, so you must DECREF it after using the value (except the one place where you INCREF it unnecessarily)

- PyErr_BadArgument() is a poor diagnostic for extend() with an array of the wrong kind.

Please fix the patch and resubmit!
-------------------------------------------------------

-------------------------------------------------------
For more info, visit:

http://sourceforge.net/patch/?func=detailpatch&patch_id=100884&group_id=5470