From 18d7d7a2176e169942d5764516eee6f952610884 Mon Sep 17 00:00:00 2001 From: Benjamin Peterson Date: Fri, 29 Jul 2011 18:27:44 -0500 Subject: also make NotImplementedType callable --- Lib/test/test_builtin.py | 2 +- Misc/NEWS | 4 ++-- Objects/object.c | 35 +++++++++++++++++++++++++++++++++++ 3 files changed, 38 insertions(+), 3 deletions(-) diff --git a/Lib/test/test_builtin.py b/Lib/test/test_builtin.py index 145ae69..4e33c23 100644 --- a/Lib/test/test_builtin.py +++ b/Lib/test/test_builtin.py @@ -1344,7 +1344,7 @@ class BuiltinTest(unittest.TestCase): self.assertRaises(TypeError, x.translate, b"1"*256, 1) def test_construct_singletons(self): - for const in None, Ellipsis: + for const in None, Ellipsis, NotImplemented: tp = type(const) self.assertIs(tp(), const) self.assertRaises(TypeError, tp, 1, 2) diff --git a/Misc/NEWS b/Misc/NEWS index d0f203a..cdbb09b 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -10,8 +10,8 @@ What's New in Python 3.3 Alpha 1? Core and Builtins ----------------- -- Make type(None) and type(Ellipsis) callable. They return the respective - singleton instances. +- Make type(None), type(Ellipsis), and type(NotImplemented) callable. They + return the respective singleton instances. - Forbid summing bytes in sum(). diff --git a/Objects/object.c b/Objects/object.c index f2f4363..cf10f3c 100644 --- a/Objects/object.c +++ b/Objects/object.c @@ -1385,6 +1385,17 @@ NotImplemented_repr(PyObject *op) return PyUnicode_FromString("NotImplemented"); } +static PyObject * +notimplemented_new(PyTypeObject *type, PyObject *args, PyObject *kwargs) +{ + if (PyTuple_GET_SIZE(args) || (kwargs && PyDict_Size(kwargs))) { + PyErr_SetString(PyExc_TypeError, "NotImplementedType takes no arguments"); + return NULL; + } + Py_INCREF(Py_NotImplemented); + return Py_NotImplemented; +} + static PyTypeObject PyNotImplemented_Type = { PyVarObject_HEAD_INIT(&PyType_Type, 0) "NotImplementedType", @@ -1400,6 +1411,30 @@ static PyTypeObject PyNotImplemented_Type = { 0, /*tp_as_sequence*/ 0, /*tp_as_mapping*/ 0, /*tp_hash */ + 0, /*tp_call */ + 0, /*tp_str */ + 0, /*tp_getattro */ + 0, /*tp_setattro */ + 0, /*tp_as_buffer */ + Py_TPFLAGS_DEFAULT, /*tp_flags */ + 0, /*tp_doc */ + 0, /*tp_traverse */ + 0, /*tp_clear */ + 0, /*tp_richcompare */ + 0, /*tp_weaklistoffset */ + 0, /*tp_iter */ + 0, /*tp_iternext */ + 0, /*tp_methods */ + 0, /*tp_members */ + 0, /*tp_getset */ + 0, /*tp_base */ + 0, /*tp_dict */ + 0, /*tp_descr_get */ + 0, /*tp_descr_set */ + 0, /*tp_dictoffset */ + 0, /*tp_init */ + 0, /*tp_alloc */ + notimplemented_new, /*tp_new */ }; PyObject _Py_NotImplementedStruct = { -- cgit v0.12