Issue 5137: SystemError when len returns a non-number (original) (raw)

This issue has been migrated to GitHub: https://github.com/python/cpython/issues/49387

classification

process

Created on 2009-02-03 04:45 by rhettinger, last changed 2022-04-11 14:56 by admin. This issue is now closed.

Messages (8)
msg81024 - (view)	Author: Raymond Hettinger (rhettinger) *	Date: 2009-02-03 04:45
len(obj) is not happy when obj.__len__() returns a non-number.
msg81338 - (view)	Author: Benjamin Peterson (benjamin.peterson) *	Date: 2009-02-07 16:58
Here's patch that raises a TypeError like 2.x.
msg81386 - (view)	Author: Antoine Pitrou (pitrou) *	Date: 2009-02-08 16:03
Wouldn't it be better to use PyNumber_AsSsize_t() instead, so as to support all __index__-enabled objects?
msg81393 - (view)	Author: Benjamin Peterson (benjamin.peterson) *	Date: 2009-02-08 17:34
Yes, good idea.
msg81395 - (view)	Author: Antoine Pitrou (pitrou) *	Date: 2009-02-08 17:42
I don't think you need to call PyNumber_Check() (unless you want to customize the error message).
msg81402 - (view)	Author: Antoine Pitrou (pitrou) *	Date: 2009-02-08 18:20
The patch is ok to me. (I assume all tests pass fine :-))
msg81418 - (view)	Author: Benjamin Peterson (benjamin.peterson) *	Date: 2009-02-08 21:07
Thanks for the reviews! Fixed in r69451.
msg81531 - (view)	Author: Hagen Fürstenau (hagen)	Date: 2009-02-10 06:55
This also solves issue 3729.