_PyEval_EvalCodeWithName(): remove redundant check

vstinner · vstinner · commit 231d1f3439f3 · 2017-01-11T02:12:06.000+01:00
Replace the runtime check with an assertion (just in case).
diff --git a/Python/ceval.c b/Python/ceval.c
@@ -4017,7 +4017,8 @@ _PyEval_EvalCodeWithName(PyObject *_co, PyObject *globals, PyObject *locals,
             }
         }
 
-        if (j >= total_args && kwdict == NULL) {
+        assert(j >= total_args);
+        if (kwdict == NULL) {
             PyErr_Format(PyExc_TypeError,
                          "%U() got an unexpected keyword argument '%S'",
                          co->co_name, keyword);

Original file line number	Diff line number	Diff line change
`@@ -4017,7 +4017,8 @@ _PyEval_EvalCodeWithName(PyObject _co, PyObject globals, PyObject *locals,`
`4017`	`4017`	`}`
`4018`	`4018`	`}`
`4019`	`4019`
`4020`		`- if (j >= total_args && kwdict == NULL) {`
	`4020`	`+ assert(j >= total_args);`
	`4021`	`+ if (kwdict == NULL) {`
`4021`	`4022`	`PyErr_Format(PyExc_TypeError,`
`4022`	`4023`	`"%U() got an unexpected keyword argument '%S'",`
`4023`	`4024`	`co->co_name, keyword);`