bpo-12159: document sys.maxsize limit in len() function reference#17934
bpo-12159: document sys.maxsize limit in len() function reference#17934vstinner merged 1 commit intopython:masterpython/cpython:masterfrom Zac-HD:document-len-overflowCopy head branch name to clipboard
Conversation
|
I think that this is a good addition to the documentation, but the example with |
There was a problem hiding this comment.
| on inputs larger than :data:`sys.maxsize`, such as :class:`range`. | |
| on lengths larger than :data:`sys.maxsize`. |
I don't understand the "such as range()" part. len(range(5)) returns 5, it doesn't fail with overflow?
There was a problem hiding this comment.
I've clarified that this is for cases such as range(2 ** 64).
sys.maxsize limit in function reference for len()
vstinner
left a comment
There was a problem hiding this comment.
LGTM, but I would prefer to get a second review. @csabella @serhiy-storchaka @JulienPalard: What do you think of this len note?
|
A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated. Once you have made the requested changes, please leave a comment on this pull request containing the phrase |
|
I have made the requested changes; please review again |
See:
__len__method but notlen(), andhttps://bugs.python.org/issue12159