Skip to content

zend_language_parser: Remove unused backup_lex_pos - #23224

Merged
iluuu1994 merged 1 commit into
php:masterfrom
ydah:remove-backup-lex-pos
Aug 12, 2026
Merged

zend_language_parser: Remove unused backup_lex_pos#23224
iluuu1994 merged 1 commit into
php:masterfrom
ydah:remove-backup-lex-pos

Conversation

@ydah

@ydah ydah commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

backup_lex_pos is an empty grammar production used by the arrow function rule to save LANG_SCNG(yy_text), but its semantic value is never consumed.

Remove the unused production and adjust the semantic value references in the arrow function rule accordingly. No runtime behavior is intended to change.

@iluuu1994 iluuu1994 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.

Indeed, leftover after removing lex_pos in GH-10046, nice catch!

@iluuu1994
iluuu1994 merged commit e36a123 into php:master Aug 12, 2026
18 checks passed
@iluuu1994

Copy link
Copy Markdown
Member

Thank you @ydah!

@ydah
ydah deleted the remove-backup-lex-pos branch August 12, 2026 13:54
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