Address code review

corona10 · Jan 26, 2024 · 9239cdc · 9239cdc
1 parent 4f7d921
commit 9239cdc
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 0 deletions.
diff --git a/Objects/floatobject.c b/Objects/floatobject.c
@@ -2016,7 +2016,9 @@ _PyFloat_Fini(_PyFreeListState *state)
     // With Py_GIL_DISABLED:
     // See `_Py_ClearFreeLists()` about why we only need to clear the freelists
     // for the current thread state.
+#ifndef Py_GIL_DISABLED
     _PyFloat_ClearFreeList(state, 1);
+#endif
 }
 
 void

diff --git a/Objects/genobject.c b/Objects/genobject.c
@@ -1688,7 +1688,9 @@ _PyAsyncGen_Fini(_PyFreeListState *state)
     // With Py_GIL_DISABLED:
     // See `_Py_ClearFreeLists()` about why we only need to clear the freelists
     // for the current thread state.
+#ifndef Py_GIL_DISABLED
     _PyAsyncGen_ClearFreeLists(state, 1);
+#endif
 }
 
 

diff --git a/Objects/listobject.c b/Objects/listobject.c
@@ -142,7 +142,9 @@ _PyList_Fini(_PyFreeListState *state)
     // With Py_GIL_DISABLED:
     // See `_Py_ClearFreeLists()` about why we only need to clear the freelists
     // for the current thread state.
+#ifndef Py_GIL_DISABLED
     _PyList_ClearFreeList(state, 1);
+#endif
 }
 
 /* Print summary info about the state of the optimized allocator */

diff --git a/Python/context.c b/Python/context.c
@@ -1290,7 +1290,9 @@ _PyContext_Fini(_PyFreeListState *state)
     // With Py_GIL_DISABLED:
     // See `_Py_ClearFreeLists()` about why we only need to clear the freelists
     // for the current thread state.
+#ifndef Py_GIL_DISABLED
     _PyContext_ClearFreeList(state, 1);
+#endif
 }