[Python-checkins] cpython: Fix struct sizes. Drop -1, since the resulting string was actually the largest
martin.v.loewis
python-checkins at python.org
Wed Sep 28 10:03:35 CEST 2011
http://hg.python.org/cpython/rev/f19af08eef02
changeset: 72480:f19af08eef02
user: Martin v. Löwis <martin at v.loewis.de>
date: Wed Sep 28 10:03:28 2011 +0200
summary:
Fix struct sizes. Drop -1, since the resulting string was actually the largest one
that could be allocated.
files:
Lib/test/test_unicode.py | 13 +++++++------
1 files changed, 7 insertions(+), 6 deletions(-)
diff --git a/Lib/test/test_unicode.py b/Lib/test/test_unicode.py
--- a/Lib/test/test_unicode.py
+++ b/Lib/test/test_unicode.py
@@ -1585,12 +1585,12 @@
def test_raiseMemError(self):
if struct.calcsize('P') == 8:
# 64 bits pointers
- ascii_struct_size = 64
- compact_struct_size = 88
+ ascii_struct_size = 48
+ compact_struct_size = 72
else:
# 32 bits pointers
- ascii_struct_size = 32
- compact_struct_size = 44
+ ascii_struct_size = 24
+ compact_struct_size = 36
for char in ('a', '\xe9', '\u20ac', '\U0010ffff'):
code = ord(char)
@@ -1604,8 +1604,9 @@
char_size = 4 # sizeof(Py_UCS4)
struct_size = compact_struct_size
# Note: sys.maxsize is half of the actual max allocation because of
- # the signedness of Py_ssize_t. -1 because of the null character.
- maxlen = ((sys.maxsize - struct_size) // char_size) - 1
+ # the signedness of Py_ssize_t. Strings of maxlen-1 should in principle
+ # be allocatable, given enough memory.
+ maxlen = ((sys.maxsize - struct_size) // char_size)
alloc = lambda: char * maxlen
self.assertRaises(MemoryError, alloc)
self.assertRaises(MemoryError, alloc)
--
Repository URL: http://hg.python.org/cpython
More information about the Python-checkins
mailing list