summaryrefslogtreecommitdiff
path: root/Python
diff options
context:
space:
mode:
authorVictor Stinner <victor.stinner@gmail.com>2016-08-22 23:17:30 +0200
committerVictor Stinner <victor.stinner@gmail.com>2016-08-22 23:17:30 +0200
commit02a024beaef0a0ae5b8cc2151ba0314fd58f6fe4 (patch)
treeb0b8b94054307dd77b0e42fd5c9f622c262f6824 /Python
parent9dae6156b5e3127cb8bfaa1bd36cecadaad5d60b (diff)
downloadcpython-02a024beaef0a0ae5b8cc2151ba0314fd58f6fe4.tar.gz
Issue #27809: Cleanup _PyEval_EvalCodeWithName()
* Rename nm to name * PEP 7: add { ... } to if/else blocks
Diffstat (limited to 'Python')
-rw-r--r--Python/ceval.c28
1 files changed, 13 insertions, 15 deletions
diff --git a/Python/ceval.c b/Python/ceval.c
index d656fab3ee..9c6593791f 100644
--- a/Python/ceval.c
+++ b/Python/ceval.c
@@ -3878,28 +3878,28 @@ _PyEval_EvalCodeWithName(PyObject *_co, PyObject *globals, PyObject *locals,
normally interned this should almost always hit. */
co_varnames = ((PyTupleObject *)(co->co_varnames))->ob_item;
for (j = 0; j < total_args; j++) {
- PyObject *nm = co_varnames[j];
- if (nm == keyword)
+ PyObject *name = co_varnames[j];
+ if (name == keyword) {
goto kw_found;
+ }
}
/* Slow fallback, just in case */
for (j = 0; j < total_args; j++) {
- PyObject *nm = co_varnames[j];
- int cmp = PyObject_RichCompareBool(
- keyword, nm, Py_EQ);
- if (cmp > 0)
+ PyObject *name = co_varnames[j];
+ int cmp = PyObject_RichCompareBool( keyword, name, Py_EQ);
+ if (cmp > 0) {
goto kw_found;
- else if (cmp < 0)
+ }
+ else if (cmp < 0) {
goto fail;
+ }
}
if (j >= total_args && kwdict == NULL) {
PyErr_Format(PyExc_TypeError,
- "%U() got an unexpected "
- "keyword argument '%S'",
- co->co_name,
- keyword);
+ "%U() got an unexpected keyword argument '%S'",
+ co->co_name, keyword);
goto fail;
}
@@ -3911,10 +3911,8 @@ _PyEval_EvalCodeWithName(PyObject *_co, PyObject *globals, PyObject *locals,
kw_found:
if (GETLOCAL(j) != NULL) {
PyErr_Format(PyExc_TypeError,
- "%U() got multiple "
- "values for argument '%S'",
- co->co_name,
- keyword);
+ "%U() got multiple values for argument '%S'",
+ co->co_name, keyword);
goto fail;
}
Py_INCREF(value);