diff -r f44f44b14dfc Objects/listobject.c --- a/Objects/listobject.c Fri Jan 20 08:35:18 2017 +0200 +++ b/Objects/listobject.c Fri Jan 20 08:54:15 2017 +0200 @@ -1912,7 +1912,7 @@ reverse_sortslice(sortslice *s, Py_ssize * duplicated). */ static PyObject * -listsort(PyListObject *self, PyObject *args, PyObject *kwds) +listsort_impl(PyListObject *self, PyObject *keyfunc, int reverse) { MergeState ms; Py_ssize_t nremaining; @@ -1922,24 +1922,11 @@ listsort(PyListObject *self, PyObject *a PyObject **saved_ob_item; PyObject **final_ob_item; PyObject *result = NULL; /* guilty until proved innocent */ - int reverse = 0; - PyObject *keyfunc = NULL; Py_ssize_t i; - static char *kwlist[] = {"key", "reverse", 0}; PyObject **keys; assert(self != NULL); assert (PyList_Check(self)); - if (args != NULL) { - if (!PyArg_ParseTupleAndKeywords(args, kwds, "|Oi:sort", - kwlist, &keyfunc, &reverse)) - return NULL; - if (Py_SIZE(args) > 0) { - PyErr_SetString(PyExc_TypeError, - "must use keyword argument for key function"); - return NULL; - } - } if (keyfunc == Py_None) keyfunc = NULL; @@ -2088,6 +2075,19 @@ keyfunc_fail: #undef IFLT #undef ISLT +static PyObject * +listsort(PyListObject *self, PyObject *args, PyObject *kwds) +{ + static char *kwlist[] = {"key", "reverse", 0}; + PyObject *keyfunc = NULL; + int reverse = 0; + + if (!PyArg_ParseTupleAndKeywords(args, kwds, "|$Oi:sort", + kwlist, &keyfunc, &reverse)) + return NULL; + return listsort_impl(self, keyfunc, reverse); +} + int PyList_Sort(PyObject *v) { @@ -2095,7 +2095,7 @@ PyList_Sort(PyObject *v) PyErr_BadInternalCall(); return -1; } - v = listsort((PyListObject *)v, (PyObject *)NULL, (PyObject *)NULL); + v = listsort_impl((PyListObject *)v, NULL, 0); if (v == NULL) return -1; Py_DECREF(v); diff -r f44f44b14dfc Python/bltinmodule.c --- a/Python/bltinmodule.c Fri Jan 20 08:35:18 2017 +0200 +++ b/Python/bltinmodule.c Fri Jan 20 08:54:15 2017 +0200 @@ -2126,15 +2126,10 @@ PyDoc_STRVAR(builtin_sorted__doc__, static PyObject * builtin_sorted(PyObject *self, PyObject **args, Py_ssize_t nargs, PyObject *kwnames) { - PyObject *newlist, *v, *seq, *keyfunc=NULL; - PyObject *callable; - static const char * const kwlist[] = {"", "key", "reverse", 0}; - /* args 1-3 should match listsort in Objects/listobject.c */ - static _PyArg_Parser parser = {"O|Oi:sorted", kwlist, 0}; - int reverse; - - if (!_PyArg_ParseStackAndKeywords(args, nargs, kwnames, &parser, - &seq, &keyfunc, &reverse)) + PyObject *newlist, *seq; + PyObject *callable, *v; + + if (!_PyArg_UnpackStack(args, nargs, "sorted", 1, 1, &seq)) return NULL; newlist = PySequence_List(seq);