You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
list.sort.__doc__ says "cmp" is a keyword, but it isn't. #46538
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
assignee='https://github.com/birkenfeld'closed_at=<Date2008-03-14.13:29:09.555>created_at=<Date2008-03-14.11:11:57.758>labels= ['docs']
title='list.sort.__doc__ says "cmp" is a keyword, but it isn\'t.'updated_at=<Date2008-03-14.13:29:09.539>user='https://bugs.python.org/dbinger'
i'm using the lastest version from subversion (trunk) and cmp still is a
keyord for list.sort.
dbinger commented on Mar 14, 2008
dbingermannequin
MannequinAuthor
More actions
Hi Bruno,
Are you testing py3k? This is what I see:
Python 3.0a3+ (py3k:61352M, Mar 12 2008, 13:11:35)
[GCC 4.0.1 (Apple Inc. build 5465)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> [].sort(cmp=lambda x, y: 1)
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
TypeError: 'cmp' is an invalid keyword argument for this function
>>>
In listobject.c, in listsort(), at line 1863, we have
the following, which appears to be where "key" and "reverse"
keyword args are supported, but not "cmp".
staticchar*kwlist[] = {"key", "reverse", 0};
assert(self!=NULL);
assert (PyList_Check(self));
if (args!=NULL) {
if (!PyArg_ParseTupleAndKeywords(args, kwds, "|Oi:sort",
Activity
dbinger commentedon Mar 14, 2008
(at revision 61376)
It looks like this docstring needs to be updated.
brunogola commentedon Mar 14, 2008
i'm using the lastest version from subversion (trunk) and cmp still is a
keyord for list.sort.
dbinger commentedon Mar 14, 2008
Hi Bruno,
Are you testing py3k? This is what I see:
In listobject.c, in listsort(), at line 1863, we have
the following, which appears to be where "key" and "reverse"
keyword args are supported, but not "cmp".
On Mar 14, 2008, at 8:41 AM, Bruno Gola wrote:
birkenfeld commentedon Mar 14, 2008
Thanks, fixed in r61377.