Fix issue 34551 - remove redundant store (#9009)

Eric Lippert · ambv · commit 5a95ba29da7e · 2018-10-22T16:52:46.000+01:00
The assignment of i/2 to nk is redundant because on this code path, nk is already the size of the dictionary, and i is already twice the size of the dictionary. I've replaced the store with an assertion that i/2 is nk.
diff --git a/Objects/call.c b/Objects/call.c
@@ -352,7 +352,7 @@ _PyFunction_FastCallDict(PyObject *func, PyObject *const *args, Py_ssize_t nargs
             Py_INCREF(k[i+1]);
             i += 2;
         }
-        nk = i / 2;
+        assert(i / 2 == nk);
     }
     else {
         kwtuple = NULL;

Original file line number	Diff line number	Diff line change
`@@ -352,7 +352,7 @@ _PyFunction_FastCallDict(PyObject func, PyObject const *args, Py_ssize_t nargs`
`352`	`352`	`Py_INCREF(k[i+1]);`
`353`	`353`	`i += 2;`
`354`	`354`	`}`
`355`		`- nk = i / 2;`
	`355`	`+ assert(i / 2 == nk);`
`356`	`356`	`}`
`357`	`357`	`else {`
`358`	`358`	`kwtuple = NULL;`