changeset:   94937:f9afa4f87570
branch:      3.4
parent:      94935:bf2016a1911f
user:        Antoine Pitrou <solipsis@pitrou.net>
date:        Tue Mar 10 22:32:00 2015 +0100
files:       Lib/test/test_buffer.py Lib/test/test_sys.py Misc/NEWS Objects/bytesobject.c Objects/tupleobject.c Objects/typeobject.c
description:
Issue #23629: Fix the default __sizeof__ implementation for variable-sized objects.


diff -r bf2016a1911f -r f9afa4f87570 Lib/test/test_buffer.py
--- a/Lib/test/test_buffer.py	Tue Mar 10 13:17:21 2015 -0700
+++ b/Lib/test/test_buffer.py	Tue Mar 10 22:32:00 2015 +0100
@@ -2449,6 +2449,21 @@
             self.assertEqual(m.tobytes(), b'')
             self.assertEqual(m.tolist(), [])
 
+    check_sizeof = support.check_sizeof
+
+    def test_memoryview_sizeof(self):
+        check = self.check_sizeof
+        vsize = support.calcvobjsize
+        base_struct = 'Pnin 2P2n2i5P 3cP'
+        per_dim = '3n'
+
+        items = list(range(8))
+        check(memoryview(b''), vsize(base_struct + 1 * per_dim))
+        a = ndarray(items, shape=[2, 4], format="b")
+        check(memoryview(a), vsize(base_struct + 2 * per_dim))
+        a = ndarray(items, shape=[2, 2, 2], format="b")
+        check(memoryview(a), vsize(base_struct + 3 * per_dim))
+
     def test_memoryview_struct_module(self):
 
         class INT(object):
diff -r bf2016a1911f -r f9afa4f87570 Lib/test/test_sys.py
--- a/Lib/test/test_sys.py	Tue Mar 10 13:17:21 2015 -0700
+++ b/Lib/test/test_sys.py	Tue Mar 10 22:32:00 2015 +0100
@@ -778,6 +778,9 @@
             check(x, vsize('n2Pi') + x.__alloc__())
         # bytearray_iterator
         check(iter(bytearray()), size('nP'))
+        # bytes
+        check(b'', vsize('n') + 1)
+        check(b'x' * 10, vsize('n') + 11)
         # cell
         def get_cell():
             x = 42
@@ -897,8 +900,6 @@
         check(int(PyLong_BASE), vsize('') + 2*self.longdigit)
         check(int(PyLong_BASE**2-1), vsize('') + 2*self.longdigit)
         check(int(PyLong_BASE**2), vsize('') + 3*self.longdigit)
-        # memoryview
-        check(memoryview(b''), size('Pnin 2P2n2i5P 3cPn'))
         # module
         check(unittest, size('PnPPP'))
         # None
diff -r bf2016a1911f -r f9afa4f87570 Misc/NEWS
--- a/Misc/NEWS	Tue Mar 10 13:17:21 2015 -0700
+++ b/Misc/NEWS	Tue Mar 10 22:32:00 2015 +0100
@@ -10,6 +10,9 @@
 Core and Builtins
 -----------------
 
+- Issue #23629: Fix the default __sizeof__ implementation for variable-sized
+  objects.
+
 Library
 -------
 
diff -r bf2016a1911f -r f9afa4f87570 Objects/bytesobject.c
--- a/Objects/bytesobject.c	Tue Mar 10 13:17:21 2015 -0700
+++ b/Objects/bytesobject.c	Tue Mar 10 22:32:00 2015 +0100
@@ -2370,18 +2370,6 @@
     return NULL;
 }
 
-PyDoc_STRVAR(sizeof__doc__,
-"B.__sizeof__() -> size of B in memory, in bytes");
-
-static PyObject *
-bytes_sizeof(PyBytesObject *v)
-{
-    Py_ssize_t res;
-    res = PyBytesObject_SIZE + Py_SIZE(v) * Py_TYPE(v)->tp_itemsize;
-    return PyLong_FromSsize_t(res);
-}
-
-
 static PyObject *
 bytes_getnewargs(PyBytesObject *v)
 {
@@ -2447,8 +2435,6 @@
      translate__doc__},
     {"upper", (PyCFunction)stringlib_upper, METH_NOARGS, _Py_upper__doc__},
     {"zfill", (PyCFunction)stringlib_zfill, METH_VARARGS, zfill__doc__},
-    {"__sizeof__", (PyCFunction)bytes_sizeof, METH_NOARGS,
-     sizeof__doc__},
     {NULL,     NULL}                         /* sentinel */
 };
 
diff -r bf2016a1911f -r f9afa4f87570 Objects/tupleobject.c
--- a/Objects/tupleobject.c	Tue Mar 10 13:17:21 2015 -0700
+++ b/Objects/tupleobject.c	Tue Mar 10 22:32:00 2015 +0100
@@ -759,27 +759,15 @@
 
 }
 
-static PyObject *
-tuple_sizeof(PyTupleObject *self)
-{
-    Py_ssize_t res;
-
-    res = PyTuple_Type.tp_basicsize + Py_SIZE(self) * sizeof(PyObject *);
-    return PyLong_FromSsize_t(res);
-}
-
 PyDoc_STRVAR(index_doc,
 "T.index(value, [start, [stop]]) -> integer -- return first index of value.\n"
 "Raises ValueError if the value is not present."
 );
 PyDoc_STRVAR(count_doc,
 "T.count(value) -> integer -- return number of occurrences of value");
-PyDoc_STRVAR(sizeof_doc,
-"T.__sizeof__() -- size of T in memory, in bytes");
 
 static PyMethodDef tuple_methods[] = {
     {"__getnewargs__",          (PyCFunction)tuple_getnewargs,  METH_NOARGS},
-    {"__sizeof__",      (PyCFunction)tuple_sizeof, METH_NOARGS, sizeof_doc},
     {"index",           (PyCFunction)tupleindex,  METH_VARARGS, index_doc},
     {"count",           (PyCFunction)tuplecount,  METH_O, count_doc},
     {NULL,              NULL}           /* sentinel */
diff -r bf2016a1911f -r f9afa4f87570 Objects/typeobject.c
--- a/Objects/typeobject.c	Tue Mar 10 13:17:21 2015 -0700
+++ b/Objects/typeobject.c	Tue Mar 10 22:32:00 2015 +0100
@@ -4258,7 +4258,7 @@
     res = 0;
     isize = self->ob_type->tp_itemsize;
     if (isize > 0)
-        res = Py_SIZE(self->ob_type) * isize;
+        res = Py_SIZE(self) * isize;
     res += self->ob_type->tp_basicsize;
 
     return PyLong_FromSsize_t(res);