Issue1337

classification

Title:	Add abstract PySequence.size() method
Type:		Severity:	normal
Components:	Core	Versions:

process

Status:	open	Resolution:
Dependencies:		Superseder:
Assigned To:		Nosy List:	pjenvey
Priority:	low	Keywords:

Created on 2009-05-03.21:51:21 by pjenvey, last changed 2009-05-03.21:51:21 by pjenvey.

Messages
msg4630 (view)	Author: Philip Jenvey (pjenvey)	Date: 2009-05-03.21:51:20
In some places PySequence (and IIRC some other places) assume __len__ returns the real length of a sequence. That's not always the case for derived objects. This is bound to break something at some point Most places (outside of PySequence) use the real size when they should (e.g. PyList/Tuple/String call size/length on themselves instead of __len__ when needed) but the base PySequence doesn't have that luxury. It should have a size() method it can rely on

msg4630 (view)

Author: Philip Jenvey (pjenvey)

Date: 2009-05-03.21:51:20

In some places PySequence (and IIRC some other places) assume __len__ 
returns the real length of a sequence. That's not always the case for 
derived objects. This is bound to break something at some point

Most places (outside of PySequence) use the real size when they should 
(e.g. PyList/Tuple/String call size/length on themselves instead of 
__len__ when needed) but the base PySequence doesn't have that luxury. 
It should have a size() method it can rely on

History
Date	User	Action	Args
2009-05-03 21:51:21	pjenvey	create