reflection: Use RETURN_COPY_VALUE()

ndossche · ndossche · commit b64cd429ca4f · 2025-11-30T01:38:53.000-08:00
diff --git a/ext/reflection/php_reflection.c b/ext/reflection/php_reflection.c
@@ -2144,7 +2144,7 @@ ZEND_METHOD(ReflectionFunction, invoke)
 	if (Z_ISREF(retval)) {
 		zend_unwrap_reference(&retval);
 	}
-	ZVAL_COPY_VALUE(return_value, &retval);
+	RETURN_COPY_VALUE(&retval);
 }
 /* }}} */
 
@@ -2177,7 +2177,7 @@ ZEND_METHOD(ReflectionFunction, invokeArgs)
 	if (Z_ISREF(retval)) {
 		zend_unwrap_reference(&retval);
 	}
-	ZVAL_COPY_VALUE(return_value, &retval);
+	RETURN_COPY_VALUE(&retval);
 }
 /* }}} */
 
@@ -3487,7 +3487,7 @@ static void reflection_method_invoke(INTERNAL_FUNCTION_PARAMETERS, int variadic)
 	if (Z_ISREF(retval)) {
 		zend_unwrap_reference(&retval);
 	}
-	ZVAL_COPY_VALUE(return_value, &retval);
+	RETURN_COPY_VALUE(&retval);
 }
 /* }}} */
 

Original file line number	Diff line number	Diff line change
`@@ -2144,7 +2144,7 @@ ZEND_METHOD(ReflectionFunction, invoke)`
`2144`	`2144`	`if (Z_ISREF(retval)) {`
`2145`	`2145`	`zend_unwrap_reference(&retval);`
`2146`	`2146`	`}`
`2147`		`- ZVAL_COPY_VALUE(return_value, &retval);`
	`2147`	`+ RETURN_COPY_VALUE(&retval);`
`2148`	`2148`	`}`
`2149`	`2149`	`/* }}} */`
`2150`	`2150`
`@@ -2177,7 +2177,7 @@ ZEND_METHOD(ReflectionFunction, invokeArgs)`
`2177`	`2177`	`if (Z_ISREF(retval)) {`
`2178`	`2178`	`zend_unwrap_reference(&retval);`
`2179`	`2179`	`}`
`2180`		`- ZVAL_COPY_VALUE(return_value, &retval);`
	`2180`	`+ RETURN_COPY_VALUE(&retval);`
`2181`	`2181`	`}`
`2182`	`2182`	`/* }}} */`
`2183`	`2183`
`@@ -3487,7 +3487,7 @@ static void reflection_method_invoke(INTERNAL_FUNCTION_PARAMETERS, int variadic)`
`3487`	`3487`	`if (Z_ISREF(retval)) {`
`3488`	`3488`	`zend_unwrap_reference(&retval);`
`3489`	`3489`	`}`
`3490`		`- ZVAL_COPY_VALUE(return_value, &retval);`
	`3490`	`+ RETURN_COPY_VALUE(&retval);`
`3491`	`3491`	`}`
`3492`	`3492`	`/* }}} */`
`3493`	`3493`