Commit ffbecc75 authored by Stefan Behnel's avatar Stefan Behnel

Remove redundant pointer comparison (already done as a shortcut before this loop).

parent 3a4fe5ec
...@@ -276,12 +276,13 @@ static int __Pyx_ParseOptionalKeywords( ...@@ -276,12 +276,13 @@ static int __Pyx_ParseOptionalKeywords(
#endif #endif
if (likely(PyUnicode_Check(key))) { if (likely(PyUnicode_Check(key))) {
while (*name) { while (*name) {
int cmp = (**name == key) ? 0 : int cmp = (
#if !CYTHON_COMPILING_IN_PYPY && PY_MAJOR_VERSION >= 3 #if !CYTHON_COMPILING_IN_PYPY && PY_MAJOR_VERSION >= 3
(__Pyx_PyUnicode_GET_LENGTH(**name) != __Pyx_PyUnicode_GET_LENGTH(key)) ? 1 : (__Pyx_PyUnicode_GET_LENGTH(**name) != __Pyx_PyUnicode_GET_LENGTH(key)) ? 1 :
#endif #endif
// In Py2, we may need to convert the argument name from str to unicode for comparison. // In Py2, we may need to convert the argument name from str to unicode for comparison.
PyUnicode_Compare(**name, key); PyUnicode_Compare(**name, key)
);
if (cmp < 0 && unlikely(PyErr_Occurred())) goto bad; if (cmp < 0 && unlikely(PyErr_Occurred())) goto bad;
if (cmp == 0) { if (cmp == 0) {
values[name-argnames] = value; values[name-argnames] = value;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment