From: Raymond Hettinger <rhettinger@users.noreply.github.com>
Date: Sun, 10 Nov 2019 04:28:31 +0000 (-0800)
Subject: Minor readability improvement for argument handling in itertools.repeat()  (GH-17101)
X-Git-Tag: v3.9.0a1~63
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=af46450bb97ab9bd38748e75aa849c29fdd70028;p=thirdparty%2FPython%2Fcpython.git

Minor readability improvement for argument handling in itertools.repeat()  (GH-17101)
---

diff --git a/Modules/itertoolsmodule.c b/Modules/itertoolsmodule.c
index 3d39fa2e4737..0cb472966d1f 100644
--- a/Modules/itertoolsmodule.c
+++ b/Modules/itertoolsmodule.c
@@ -4256,17 +4256,17 @@ repeat_new(PyTypeObject *type, PyObject *args, PyObject *kwds)
 {
     repeatobject *ro;
     PyObject *element;
-    Py_ssize_t cnt = -1, n_kwds = 0;
+    Py_ssize_t cnt = -1, n_args;
     static char *kwargs[] = {"object", "times", NULL};
 
+    n_args = PyTuple_GET_SIZE(args);
+    if (kwds != NULL)
+        n_args += PyDict_GET_SIZE(kwds);
     if (!PyArg_ParseTupleAndKeywords(args, kwds, "O|n:repeat", kwargs,
                                      &element, &cnt))
         return NULL;
-
-    if (kwds != NULL)
-        n_kwds = PyDict_GET_SIZE(kwds);
     /* Does user supply times argument? */
-    if ((PyTuple_Size(args) + n_kwds == 2) && cnt < 0)
+    if (n_args == 2 && cnt < 0)
         cnt = 0;
 
     ro = (repeatobject *)type->tp_alloc(type, 0);