]> granicus.if.org Git - python/commitdiff
Merged revisions 84320 via svnmerge from
authorBenjamin Peterson <benjamin@python.org>
Wed, 25 Aug 2010 23:19:30 +0000 (23:19 +0000)
committerBenjamin Peterson <benjamin@python.org>
Wed, 25 Aug 2010 23:19:30 +0000 (23:19 +0000)
svn+ssh://pythondev@svn.python.org/python/branches/py3k

........
  r84320 | benjamin.peterson | 2010-08-25 18:13:17 -0500 (Wed, 25 Aug 2010) | 1 line

  basicsize and itemsize are Py_ssize_t #9688
........

Lib/test/test_types.py
Misc/NEWS
Objects/typeobject.c

index a56476b4e7ca5fcaf153fd13fb973adc0045c7f1..d227ab1e2bcf6fdb5fad10f2dc609cb3639bd5b4 100644 (file)
@@ -666,6 +666,11 @@ class TypesTests(unittest.TestCase):
         for code in 'xXobns':
             self.assertRaises(ValueError, format, 0, ',' + code)
 
+    def test_internal_sizes(self):
+        self.assertGreater(object.__basicsize__, 0)
+        self.assertGreater(tuple.__itemsize__, 0)
+
+
 def test_main():
     run_unittest(TypesTests)
 
index 48f1b9a3743abf8af5a96934479a011b1b1bf07b..4bedeb0a53795c1c4b4e9a459618d4cc8336e57f 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -14,6 +14,8 @@ Core and Builtins
 
 - Restore GIL in nis_cat in case of error.
 
+- Issue #9688: __basicsize__ and __itemsize__ must be accessed as Py_ssize_t.
+
 - Issue #5319: Print an error if flushing stdout fails at interpreter
   shutdown.
 
index 3bf4e908fe44cc31ef2585e275760d6af83bd5f8..d2124f17c36d811987fa0b98f44a8158f40ef960 100644 (file)
@@ -189,8 +189,8 @@ assign_version_tag(PyTypeObject *type)
 
 
 static PyMemberDef type_members[] = {
-    {"__basicsize__", T_INT, offsetof(PyTypeObject,tp_basicsize),READONLY},
-    {"__itemsize__", T_INT, offsetof(PyTypeObject, tp_itemsize), READONLY},
+    {"__basicsize__", T_PYSSIZET, offsetof(PyTypeObject,tp_basicsize),READONLY},
+    {"__itemsize__", T_PYSSIZET, offsetof(PyTypeObject, tp_itemsize), READONLY},
     {"__flags__", T_LONG, offsetof(PyTypeObject, tp_flags), READONLY},
     {"__weakrefoffset__", T_LONG,
      offsetof(PyTypeObject, tp_weaklistoffset), READONLY},