fix: the bytecode for yields was only different for 3.13

nedbat · Aug 8, 2024 · c3c91f1 · c3c91f1
1 parent b9501d6
commit c3c91f1
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/coverage/ctracer/util.h b/coverage/ctracer/util.h
@@ -60,9 +60,10 @@
 #endif
 
 // Where does frame.f_lasti point when yielding from a generator?
-// It used to point at the YIELD, now it points at the RESUME.
+// It used to point at the YIELD, in 3.13 it points at the RESUME,
+// then it went back to the YIELD.
 // https://github.com/python/cpython/issues/113728
-#define ENV_LASTI_IS_YIELD (PY_VERSION_HEX < 0x030D0000)
+#define ENV_LASTI_IS_YIELD ((PY_VERSION_HEX & 0xFFFF0000) != 0x030D0000)
 
 /* The values returned to indicate ok or error. */
 #define RET_OK      0

diff --git a/coverage/env.py b/coverage/env.py
@@ -120,9 +120,10 @@ class PYBEHAVIOR:
     pep669 = bool(getattr(sys, "monitoring", None))
 
     # Where does frame.f_lasti point when yielding from a generator?
-    # It used to point at the YIELD, now it points at the RESUME.
+    # It used to point at the YIELD, in 3.13 it points at the RESUME,
+    # then it went back to the YIELD.
     # https://github.com/python/cpython/issues/113728
-    lasti_is_yield = (PYVERSION < (3, 13))
+    lasti_is_yield = (PYVERSION[:2] != (3, 13))
 
 
 # Coverage.py specifics, about testing scenarios. See tests/testenv.py also.