diff options
author | Neal Norwitz <nnorwitz@gmail.com> | 2007-02-25 19:44:48 (GMT) |
---|---|---|
committer | Neal Norwitz <nnorwitz@gmail.com> | 2007-02-25 19:44:48 (GMT) |
commit | ee3a1b5244e60566c5d5c8f6a1ea4b381de99f1c (patch) | |
tree | ad52a17fc77530870a575107a7cea9c27de17606 /Include/longobject.h | |
parent | 5a3e812444609329b30be837a08c11269d917b72 (diff) | |
download | cpython-ee3a1b5244e60566c5d5c8f6a1ea4b381de99f1c.zip cpython-ee3a1b5244e60566c5d5c8f6a1ea4b381de99f1c.tar.gz cpython-ee3a1b5244e60566c5d5c8f6a1ea4b381de99f1c.tar.bz2 |
Variation of patch # 1624059 to speed up checking if an object is a subclass
of some of the common builtin types.
Use a bit in tp_flags for each common builtin type. Check the bit
to determine if any instance is a subclass of these common types.
The check avoids a function call and O(n) search of the base classes.
The check is done in the various Py*_Check macros rather than calling
PyType_IsSubtype().
All the bits are set in tp_flags when the type is declared
in the Objects/*object.c files because PyType_Ready() is not called
for all the types. Should PyType_Ready() be called for all types?
If so and the change is made, the changes to the Objects/*object.c files
can be reverted (remove setting the tp_flags). Objects/typeobject.c
would also have to be modified to add conditions
for Py*_CheckExact() in addition to each the PyType_IsSubtype check.
Diffstat (limited to 'Include/longobject.h')
-rw-r--r-- | Include/longobject.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/Include/longobject.h b/Include/longobject.h index eef4e9b..3893ad6 100644 --- a/Include/longobject.h +++ b/Include/longobject.h @@ -11,7 +11,8 @@ typedef struct _longobject PyLongObject; /* Revealed in longintrepr.h */ PyAPI_DATA(PyTypeObject) PyLong_Type; -#define PyLong_Check(op) PyObject_TypeCheck(op, &PyLong_Type) +#define PyLong_Check(op) \ + PyType_FastSubclass((op)->ob_type, Py_TPFLAGS_LONG_SUBCLASS) #define PyLong_CheckExact(op) ((op)->ob_type == &PyLong_Type) PyAPI_FUNC(PyObject *) PyLong_FromLong(long); |