we do not expect a bailout to be something common

realFlowControl · realFlowControl · commit 5a9a24f34a7c · 2024-07-22T10:51:56.000+02:00
diff --git a/src/parallel.c b/src/parallel.c
@@ -235,7 +235,7 @@ PHP_RSHUTDOWN_FUNCTION(PARALLEL_CORE)
     // In case of a `zend_bailout()` this mutex could still be locked, so we
     // unlock it just in case.
     // See https://github.com/krakjoe/parallel/issues/313 for more details
-    if (CG(unclean_shutdown) == 1) {
+    if (UNEXPECTED(CG(unclean_shutdown) == 1)) {
         pthread_mutex_unlock(&php_parallel_output_mutex);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -235,7 +235,7 @@ PHP_RSHUTDOWN_FUNCTION(PARALLEL_CORE)`
`235`	`235`	// In case of a `zend_bailout()` this mutex could still be locked, so we
`236`	`236`	`// unlock it just in case.`
`237`	`237`	`// See https://github.com/krakjoe/parallel/issues/313 for more details`
`238`		`- if (CG(unclean_shutdown) == 1) {`
	`238`	`+ if (UNEXPECTED(CG(unclean_shutdown) == 1)) {`
`239`	`239`	`pthread_mutex_unlock(&php_parallel_output_mutex);`
`240`	`240`	`}`
`241`	`241`