]> granicus.if.org Git - python/commit
docompare(): Use PyTuple_New instead of Py_BuildValue to build compare's
authorTim Peters <tim.peters@gmail.com>
Thu, 11 Jul 2002 21:46:16 +0000 (21:46 +0000)
committerTim Peters <tim.peters@gmail.com>
Thu, 11 Jul 2002 21:46:16 +0000 (21:46 +0000)
commitf2a0473350d91065cb1ac14ca4b55d56eea1ed38
tree2e7aa3a55a29a8883ccbb629142dbe7eb0917321
parent1808fbc46daae1e8a95359716e48b3f3a927c203
docompare():  Use PyTuple_New instead of Py_BuildValue to build compare's
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.
Objects/listobject.c