From e2eda606a83cbe68846faf44b478053e250fb025 Mon Sep 17 00:00:00 2001 From: Raymond Hettinger Date: Sun, 4 Apr 2004 08:51:41 +0000 Subject: Improve accuracy of sequence and mapping checks. --- Misc/NEWS | 3 +++ Objects/abstract.c | 10 ++++++++-- 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/Misc/NEWS b/Misc/NEWS index b1f2043..c51b74b 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -202,6 +202,9 @@ Core and builtins Extension modules ----------------- +- operator.isMappingType() and operator.isSequenceType() now give + fewer false positives. + - socket.sslerror is now a subclass of socket.error . Also added socket.error to the socket module's C API. diff --git a/Objects/abstract.c b/Objects/abstract.c index bf60c75..307ef86 100644 --- a/Objects/abstract.c +++ b/Objects/abstract.c @@ -1058,6 +1058,8 @@ PyNumber_Float(PyObject *o) int PySequence_Check(PyObject *s) { + if (PyInstance_Check(s)) + return PyObject_HasAttrString(s, "__getitem__"); return s != NULL && s->ob_type->tp_as_sequence && s->ob_type->tp_as_sequence->sq_item != NULL; } @@ -1600,8 +1602,12 @@ PySequence_Index(PyObject *s, PyObject *o) int PyMapping_Check(PyObject *o) { - return o && o->ob_type->tp_as_mapping && - o->ob_type->tp_as_mapping->mp_subscript; + if (PyInstance_Check(o)) + return PyObject_HasAttrString(o, "__getitem__"); + + return o && o->ob_type->tp_as_mapping && + o->ob_type->tp_as_mapping->mp_subscript && + !PyObject_HasAttrString(o, "__getslice__"); } int -- cgit v0.12