Skip to content

ext/curl: Remove unused return_value parameter from _php_curl_multi_setopt - #24253

Merged
ndossche merged 1 commit into
php:masterfrom
arshidkv12:fix/unused
Oct 11, 2026
Merged

ndossche merged 1 commit into
php:masterfrom
arshidkv12:fix/unused

Conversation

@arshidkv12

Copy link
Copy Markdown
Contributor
-static bool _php_curl_multi_setopt(php_curlm *mh, zend_long option, zval *zvalue, zval *return_value) /* {{{ */
+static bool _php_curl_multi_setopt(php_curlm *mh, zend_long option, zval *zvalue) /* {{{ */
 {
        CURLMcode error = CURLM_OK;
 
@@ -521,7 +521,7 @@ PHP_FUNCTION(curl_multi_setopt)
 
        mh = Z_CURL_MULTI_P(z_mh);
 
-       RETURN_BOOL(_php_curl_multi_setopt(mh, options, zvalue, return_value));

@ndossche ndossche left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@ndossche
ndossche merged commit 595abc1 into php:master Oct 11, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants