commit 533a9b459b29e8c2254b6bf6d82fd1cfa83be8cd
author Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com> Thu Jan 31 15:35:00 2019 -0800
committer Raymond Hettinger <rhettinger@users.noreply.github.com> Thu Jan 31 15:35:00 2019 -0800
tree bbf62364f68a6850bbd6efa00c668b596981e9f0
parent 09b66e027bdd4d7af6921fa2a2c9536f9cf5eebe

Consistently move the misses update to just before the user function call (GH-11715) (GH-11716)

Modules/_functoolsmodule.c

diff --git a/Modules/_functoolsmodule.c b/Modules/_functoolsmodule.c
index 4aca63e..90c698e 100644
--- a/Modules/_functoolsmodule.c
+++ b/Modules/_functoolsmodule.c
@@ -796,10 +796,12 @@
 static PyObject *
 uncached_lru_cache_wrapper(lru_cache_object *self, PyObject *args, PyObject *kwds)
 {
-    PyObject *result = PyObject_Call(self->func, args, kwds);
+    PyObject *result;
+
+    self->misses++;
+    result = PyObject_Call(self->func, args, kwds);
     if (!result)
         return NULL;
-    self->misses++;
     return result;
 }
 
@@ -827,6 +829,7 @@
         Py_DECREF(key);
         return NULL;
     }
+    self->misses++;
     result = PyObject_Call(self->func, args, kwds);
     if (!result) {
         Py_DECREF(key);
@@ -838,7 +841,6 @@
         return NULL;
     }
     Py_DECREF(key);
-    self->misses++;
     return result;
 }