bpo-44434: Remove useless calls to PyThread_exit_thread() (GH-26943)

vstinner · Alexey Izbyshev · web-flow · commit 48e3a1d95aee · 2021-06-29T02:03:30.000+02:00
Remove useless calls to PyThread_exit_thread() in two unit tests of
_testcapi and _testembed modules.

Co-authored-by: Alexey Izbyshev &lt;izbyshev@ispras.ru&gt;
diff --git a/Modules/_testcapimodule.c b/Modules/_testcapimodule.c
@@ -4300,8 +4300,6 @@ temporary_c_thread(void *data)
     PyGILState_Release(state);
 
     PyThread_release_lock(test_c_thread->exit_event);
-
-    PyThread_exit_thread();
 }
 
 static PyObject *
diff --git a/Programs/_testembed.c b/Programs/_testembed.c
@@ -269,8 +269,6 @@ static void bpo20891_thread(void *lockp)
     PyGILState_Release(state);
 
     PyThread_release_lock(lock);
-
-    PyThread_exit_thread();
 }
 
 static int test_bpo20891(void)

Original file line number	Diff line number	Diff line change
`@@ -4300,8 +4300,6 @@ temporary_c_thread(void *data)`
`4300`	`4300`	`PyGILState_Release(state);`
`4301`	`4301`
`4302`	`4302`	`PyThread_release_lock(test_c_thread->exit_event);`
`4303`		`-`
`4304`		`- PyThread_exit_thread();`
`4305`	`4303`	`}`
`4306`	`4304`
`4307`	`4305`	`static PyObject *`
Original file line number	Diff line number	Diff line change
`@@ -269,8 +269,6 @@ static void bpo20891_thread(void *lockp)`
`269`	`269`	`PyGILState_Release(state);`
`270`	`270`
`271`	`271`	`PyThread_release_lock(lock);`
`272`		`-`
`273`		`- PyThread_exit_thread();`
`274`	`272`	`}`
`275`	`273`
`276`	`274`	`static int test_bpo20891(void)`