mirrors/cpython
0
0
Fork 0
mirror of https://github.com/python/cpython.git synced 2024-11-26 19:34:19 +08:00
Code Issues Packages Projects Releases Wiki Activity

docompare(): Use PyTuple_New instead of Py_BuildValue to build compare's

Browse Source 
arg tuple.  This was suggested on c.l.py but afraid I can't find the msg
again for proper attribution.  For

    list.sort(cmp)

where list is a list of random ints, and cmp is __builtin__.cmp, this
yields an overall 50-60% speedup on my Win2K box.  Of course this is a
best case, because the overhead of calling cmp relative to the cost of
actually comparing two ints is at an extreme.  Nevertheless it's huge
bang for the buck.  An additionak 20-30% can be bought by making the arg
tuple an immortal static (avoiding all but "the first" PyTuple_New), but
that's tricky to make correct since docompare needs to be reentrant.  So
this picks the cherry and leaves the pits for Fred <wink>.

Note that this makes no difference to the

    list.sort()

case; an arg tuple gets built only if the user specifies an explicit
sort function.
This commit is contained in:
Tim Peters 2002-07-11 21:46:16 +00:00
parent 1808fbc46d
commit f2a0473350
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
Objects/listobject.c
View File

@ -756,7 +756,8 @@ listpop(PyListObject *self, PyObject *args)
static int
docompare(PyObject *x, PyObject *y, PyObject *compare)
{
PyObject *args, *res;
PyObject *res;
PyObject *args;
int i;
if (compare == NULL) {
@ -772,9 +773,13 @@ docompare(PyObject *x, PyObject *y, PyObject *compare)
return -i;
}
args = Py_BuildValue("(OO)", x, y);
args = PyTuple_New(2);
if (args == NULL)
return CMPERROR;
Py_INCREF(x);
Py_INCREF(y);
PyTuple_SET_ITEM(args, 0, x);
PyTuple_SET_ITEM(args, 1, y);
res = PyEval_CallObject(compare, args);
Py_DECREF(args);
if (res == NULL)