changeset:   106365:758674087b12
user:        Victor Stinner <victor.stinner@gmail.com>
date:        Wed Feb 01 17:42:18 2017 +0100
files:       Python/getargs.c
description:
Issue #29286: Rename private PyArg_UnpackStack_impl() to unpack_stack()

Rename also "l" argument to "nargs".


diff -r fcba9c9d987b -r 758674087b12 Python/getargs.c
--- a/Python/getargs.c	Wed Feb 01 17:04:52 2017 +0100
+++ b/Python/getargs.c	Wed Feb 01 17:42:18 2017 +0100
@@ -2352,8 +2352,8 @@
 
 
 static int
-PyArg_UnpackStack_impl(PyObject **args, Py_ssize_t l, const char *name,
-                       Py_ssize_t min, Py_ssize_t max, va_list vargs)
+unpack_stack(PyObject **args, Py_ssize_t nargs, const char *name,
+             Py_ssize_t min, Py_ssize_t max, va_list vargs)
 {
     Py_ssize_t i;
     PyObject **o;
@@ -2361,41 +2361,41 @@
     assert(min >= 0);
     assert(min <= max);
 
-    if (l < min) {
+    if (nargs < min) {
         if (name != NULL)
             PyErr_Format(
                 PyExc_TypeError,
                 "%s expected %s%zd arguments, got %zd",
-                name, (min == max ? "" : "at least "), min, l);
+                name, (min == max ? "" : "at least "), min, nargs);
         else
             PyErr_Format(
                 PyExc_TypeError,
                 "unpacked tuple should have %s%zd elements,"
                 " but has %zd",
-                (min == max ? "" : "at least "), min, l);
+                (min == max ? "" : "at least "), min, nargs);
         return 0;
     }
 
-    if (l == 0) {
+    if (nargs == 0) {
         return 1;
     }
 
-    if (l > max) {
+    if (nargs > max) {
         if (name != NULL)
             PyErr_Format(
                 PyExc_TypeError,
                 "%s expected %s%zd arguments, got %zd",
-                name, (min == max ? "" : "at most "), max, l);
+                name, (min == max ? "" : "at most "), max, nargs);
         else
             PyErr_Format(
                 PyExc_TypeError,
                 "unpacked tuple should have %s%zd elements,"
                 " but has %zd",
-                (min == max ? "" : "at most "), max, l);
+                (min == max ? "" : "at most "), max, nargs);
         return 0;
     }
 
-    for (i = 0; i < l; i++) {
+    for (i = 0; i < nargs; i++) {
         o = va_arg(vargs, PyObject **);
         *o = args[i];
     }
@@ -2423,7 +2423,7 @@
 #else
     va_start(vargs);
 #endif
-    retval = PyArg_UnpackStack_impl(stack, nargs, name, min, max, vargs);
+    retval = unpack_stack(stack, nargs, name, min, max, vargs);
     va_end(vargs);
     return retval;
 }
@@ -2440,7 +2440,7 @@
 #else
     va_start(vargs);
 #endif
-    retval = PyArg_UnpackStack_impl(args, nargs, name, min, max, vargs);
+    retval = unpack_stack(args, nargs, name, min, max, vargs);
     va_end(vargs);
     return retval;
 }