Rename interpreter cleanup functions

ashm-dev · ashm-dev · commit aa48624bea67 · 2025-10-16T21:47:39.000+03:00
Rename functions to be more descriptive of their actions.
diff --git a/Python/pystate.c b/Python/pystate.c
@@ -489,19 +489,19 @@ free_interpreter(PyInterpreterState *interp)
 }
 
 static void
-_finalize_and_free_interpreter(PyInterpreterState *interp)
+cleanup_and_free_interpreter(PyInterpreterState *interp)
 {
     _Py_qsbr_fini(interp);
     _PyObject_FiniState(interp);
     free_interpreter(interp);
 }
 
 static inline void
-_interp_release_owner(PyInterpreterState *interp)
+release_interp_owner(PyInterpreterState *interp)
 {
     Py_ssize_t prev = _Py_atomic_add_ssize(&interp->owners, -1);
     if (prev == 1) {
-        _finalize_and_free_interpreter(interp);
+        cleanup_and_free_interpreter(interp);
     }
 }
 
@@ -978,7 +978,7 @@ PyInterpreterState_Delete(PyInterpreterState *interp)
 
     interp->finalizing = 1;
 
-    _interp_release_owner(interp);
+    release_interp_owner(interp);
 }
 
 
@@ -1444,7 +1444,7 @@ free_threadstate(_PyThreadStateImpl *tstate)
         PyMem_RawFree(tstate);
     }
 
-    _interp_release_owner(interp);
+    release_interp_owner(interp);
 }
 
 static void

Original file line number	Diff line number	Diff line change
`@@ -489,19 +489,19 @@ free_interpreter(PyInterpreterState *interp)`
`489`	`489`	`}`
`490`	`490`
`491`	`491`	`static void`
`492`		`-_finalize_and_free_interpreter(PyInterpreterState *interp)`
	`492`	`+cleanup_and_free_interpreter(PyInterpreterState *interp)`
`493`	`493`	`{`
`494`	`494`	`_Py_qsbr_fini(interp);`
`495`	`495`	`_PyObject_FiniState(interp);`
`496`	`496`	`free_interpreter(interp);`
`497`	`497`	`}`
`498`	`498`
`499`	`499`	`static inline void`
`500`		`-_interp_release_owner(PyInterpreterState *interp)`
	`500`	`+release_interp_owner(PyInterpreterState *interp)`
`501`	`501`	`{`
`502`	`502`	`Py_ssize_t prev = _Py_atomic_add_ssize(&interp->owners, -1);`
`503`	`503`	`if (prev == 1) {`
`504`		`- _finalize_and_free_interpreter(interp);`
	`504`	`+ cleanup_and_free_interpreter(interp);`
`505`	`505`	`}`
`506`	`506`	`}`
`507`	`507`
`@@ -978,7 +978,7 @@ PyInterpreterState_Delete(PyInterpreterState *interp)`
`978`	`978`
`979`	`979`	`interp->finalizing = 1;`
`980`	`980`
`981`		`- _interp_release_owner(interp);`
	`981`	`+ release_interp_owner(interp);`
`982`	`982`	`}`
`983`	`983`
`984`	`984`
`@@ -1444,7 +1444,7 @@ free_threadstate(_PyThreadStateImpl *tstate)`
`1444`	`1444`	`PyMem_RawFree(tstate);`
`1445`	`1445`	`}`
`1446`	`1446`
`1447`		`- _interp_release_owner(interp);`
	`1447`	`+ release_interp_owner(interp);`
`1448`	`1448`	`}`
`1449`	`1449`
`1450`	`1450`	`static void`