Skip to content

tree-wide: Generate nicer code in Z_PARAM_*() helpers wrapping Z_PARAM_*() - #23100

Merged
TimWolla merged 1 commit into
php:masterfrom
TimWolla:z-param-wrapper-macro-nicer-helper-variables
Aug 7, 2026
Merged

tree-wide: Generate nicer code in Z_PARAM_*() helpers wrapping Z_PARAM_*()#23100
TimWolla merged 1 commit into
php:masterfrom
TimWolla:z-param-wrapper-macro-nicer-helper-variables

Conversation

@TimWolla

@TimWolla TimWolla commented Aug 7, 2026

Copy link
Copy Markdown
Member

Instead of using a fixed variable name for the temporary, it is now derived from the name of the target variable.

…ARAM_*()`

Instead of using a fixed variable name for the temporary, it is now derived
from the name of the target variable.

@arnaud-lb arnaud-lb 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.

Looks good to me!

@TimWolla
TimWolla merged commit 909f722 into php:master Aug 7, 2026
18 checks passed
@TimWolla
TimWolla deleted the z-param-wrapper-macro-nicer-helper-variables branch August 7, 2026 14:09
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