summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBenjamin Peterson <benjamin@python.org>2010-10-02 00:08:58 +0000
committerBenjamin Peterson <benjamin@python.org>2010-10-02 00:08:58 +0000
commita5d5cc45c9a8991a74f111e9d4846a0c043d7674 (patch)
tree3838b4f7ba6cf4c2d92b3fb2babbad2682cfef3b
parent1390dd7c9b1f621e0c31c60547d0beb5d432fb06 (diff)
downloadcpython-git-a5d5cc45c9a8991a74f111e9d4846a0c043d7674.tar.gz
Merged revisions 85154 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/branches/py3k ........ r85154 | benjamin.peterson | 2010-10-01 19:03:31 -0500 (Fri, 01 Oct 2010) | 1 line type.__abstractmethods__ should raise an AttributeError #10006 ........
-rw-r--r--Lib/test/test_abc.py7
-rw-r--r--Misc/NEWS2
-rw-r--r--Objects/typeobject.c7
3 files changed, 14 insertions, 2 deletions
diff --git a/Lib/test/test_abc.py b/Lib/test/test_abc.py
index 6a8c3a1327..edd2c047f2 100644
--- a/Lib/test/test_abc.py
+++ b/Lib/test/test_abc.py
@@ -70,6 +70,13 @@ class TestABC(unittest.TestCase):
self.assertFalse(issubclass(OldstyleClass, A))
self.assertFalse(issubclass(A, OldstyleClass))
+ def test_type_has_no_abstractmethods(self):
+ # type pretends not to have __abstractmethods__.
+ self.assertRaises(AttributeError, getattr, type, "__abstractmethods__")
+ class meta(type):
+ pass
+ self.assertRaises(AttributeError, getattr, meta, "__abstractmethods__")
+
def test_isinstance_class(self):
class A:
__metaclass__ = abc.ABCMeta
diff --git a/Misc/NEWS b/Misc/NEWS
index 1fabf61501..94c0ff72d7 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -17,6 +17,8 @@ Core and Builtins
fixes an interpreter crash when initializing an instance of a long
subclass from an object whose __long__ method returns a plain int.
+- Issue #10006: type.__abstractmethods__ now raises an AttributeError.
+
- Issue #9797: pystate.c wrongly assumed that zero couldn't be a valid
thread-local storage key.
diff --git a/Objects/typeobject.c b/Objects/typeobject.c
index d262168120..9cb7e6238e 100644
--- a/Objects/typeobject.c
+++ b/Objects/typeobject.c
@@ -307,8 +307,11 @@ type_set_module(PyTypeObject *type, PyObject *value, void *context)
static PyObject *
type_abstractmethods(PyTypeObject *type, void *context)
{
- PyObject *mod = PyDict_GetItemString(type->tp_dict,
- "__abstractmethods__");
+ PyObject *mod = NULL;
+ /* type its self has an __abstractmethods__ descriptor (this). Don't
+ return that. */
+ if (type != &PyType_Type)
+ mod = PyDict_GetItemString(type->tp_dict, "__abstractmethods__");
if (!mod) {
PyErr_Format(PyExc_AttributeError, "__abstractmethods__");
return NULL;