commit e380e0986bcec1eff78655d559774d71417b799d
author Benjamin Peterson <benjamin@python.org> Mon Mar 17 16:00:27 2014 -0500
committer Benjamin Peterson <benjamin@python.org> Mon Mar 17 16:00:27 2014 -0500
tree 0dc9d8e047f4552d8ff3543bbdea4401a69ed349
parent 6373ba5199fe706f174aed482d77e2bb4b29e2f0
parent df813791db32867634c86c33714995dfeb651099

merge 3.4

Lib/test/test_descr.py

diff --git a/Lib/test/test_descr.py b/Lib/test/test_descr.py
index 2a9e329..71d8609 100644
--- a/Lib/test/test_descr.py
+++ b/Lib/test/test_descr.py
@@ -4414,6 +4414,14 @@
             self.assertRaises(TypeError, case, 1, 2, 3)
             self.assertRaises(TypeError, case, 1, 2, foo=3)
 
+    def test_subclassing_does_not_duplicate_dict_descriptors(self):
+        class Base:
+            pass
+        class Sub(Base):
+            pass
+        self.assertIn("__dict__", Base.__dict__)
+        self.assertNotIn("__dict__", Sub.__dict__)
+
 
 class DictProxyTests(unittest.TestCase):
     def setUp(self):

Objects/typeobject.c

diff --git a/Objects/typeobject.c b/Objects/typeobject.c
index df8f351..c21b397 100644
--- a/Objects/typeobject.c
+++ b/Objects/typeobject.c
@@ -2472,12 +2472,6 @@
             type->tp_dictoffset = slotoffset;
         slotoffset += sizeof(PyObject *);
     }
-    else if (!type->tp_dictoffset) {
-        type->tp_dictoffset = base->tp_dictoffset;
-    }
-    if (type->tp_dictoffset) {
-        et->ht_cached_keys = _PyDict_NewKeysForClass();
-    }
     if (add_weak) {
         assert(!base->tp_itemsize);
         type->tp_weaklistoffset = slotoffset;
@@ -2527,6 +2521,10 @@
     /* Put the proper slots in place */
     fixup_slot_dispatchers(type);
 
+    if (type->tp_dictoffset) {
+        et->ht_cached_keys = _PyDict_NewKeysForClass();
+    }
+
     Py_DECREF(dict);
     return (PyObject *)type;
 
@@ -2643,9 +2641,6 @@
             type->tp_doc = tp_doc;
         }
     }
-    if (type->tp_dictoffset) {
-        res->ht_cached_keys = _PyDict_NewKeysForClass();
-    }
     if (type->tp_dealloc == NULL) {
         /* It's a heap type, so needs the heap types' dealloc.
            subtype_dealloc will call the base type's tp_dealloc, if
@@ -2656,6 +2651,10 @@
     if (PyType_Ready(type) < 0)
         goto fail;
 
+    if (type->tp_dictoffset) {
+        res->ht_cached_keys = _PyDict_NewKeysForClass();
+    }
+
     /* Set type.__module__ */
     s = strrchr(spec->name, '.');
     if (s != NULL)