Skip to content

Zend: remove ZEND_RETURN_{VAL|REF|VALUE|REFERENCE} defines - #25078

Open
Girgias wants to merge 2 commits into
php:masterfrom
Girgias:2026-10-remove-unused-define
Open

Girgias wants to merge 2 commits into
php:masterfrom
Girgias:2026-10-remove-unused-define

Conversation

@Girgias

@Girgias Girgias commented Oct 31, 2026

Copy link
Copy Markdown
Member

These seem to be effectively unused and somewhat confusing at the same time.

As those are unused and seemingly conflict with ZEND_RETURN_{VALUE|REFERENCE}
As ZEND_RETURN_VALUE is used once in the ZEND_BEGIN_ARG_INFO() macro definition to pass to ZEND_BEGIN_ARG_INFO_EX().
However every other call to ZEND_BEGIN_ARG_INFO_EX() just uses 0/1 directly.
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.

1 participant