Extend pre-allocated integers to cover [0, 255]

[tcdelaney]

BPO	1436243
Nosy	@birkenfeld, @rhettinger, @terryjreedy

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = 'https://github.com/birkenfeld'
closed_at = <Date 2006-02-22.18:04:10.000>
created_at = <Date 2006-02-21.22:43:30.000>
labels = ['type-feature']
title = 'Extend pre-allocated integers to cover [0, 255]'
updated_at = <Date 2006-02-22.18:04:10.000>
user = 'https://bugs.python.org/tcdelaney'

bugs.python.org fields:

activity = <Date 2006-02-22.18:04:10.000>
actor = 'terry.reedy'
assignee = 'georg.brandl'
closed = True
closed_date = None
closer = None
components = ['None']
creation = <Date 2006-02-21.22:43:30.000>
creator = 'tcdelaney'
dependencies = []
files = []
hgrepos = []
issue_num = 1436243
keywords = []
message_count = 5.0
messages = ['54735', '54736', '54737', '54738', '54739']
nosy_count = 4.0
nosy_names = ['georg.brandl', 'rhettinger', 'terry.reedy', 'tcdelaney']
pr_nums = []
priority = 'normal'
resolution = 'accepted'
stage = None
status = 'closed'
superseder = None
type = 'enhancement'
url = 'https://bugs.python.org/issue1436243'
versions = []

[tcdelaney]

When the bytes object is introduced (successor to
PEP-332) the full range of byte integer objects [0,
255] are likely to be used more commonly.

Suggest extending the pre-allocated integer obejcts to
include the full range [0, 255].

[rhettinger]

Logged In: YES
user_id=80475

Georg, would you take care of this one for Py2.5.
Also include the value 256.

[terryjreedy]

Logged In: YES
user_id=593130

I decided to test out the new svn view facility ;-)

Looks like 100 in
#define NSMALLPOSINTS		100
in python/trunk/Objects/intobject.c
just needs to be upped to 257

Unlike for other builtin types, I could not find a
Lib/test/test_int.py file or equivalent where ints are
tested, to see if pre-allocation is tested. What did I
miss?

[birkenfeld]

Logged In: YES
user_id=849994

It's test_types, and I added a test too.

Revision 42552.

[terryjreedy]

Logged In: YES
user_id=593130

Thanks for the info.  Small nit: your checkin message says
"RFE python/cpython#42933: make integers in [0..256] preallocated. "
The range is actually (now)[-5...256].
#define NSMALLNEGINTS		5
sets the lower limit (unchanged).