Skip to content

zend_ast: Fix ast export del in names, invalid variable and property names - #25227

Open
jorgsowa wants to merge 2 commits into
php:PHP-8.4from
jorgsowa:fix-ast-export-del-in-names
Open

jorgsowa wants to merge 2 commits into
php:PHP-8.4from
jorgsowa:fix-ast-export-del-in-names

Conversation

@jorgsowa

@jorgsowa jorgsowa commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

Fixes two AST export bugs that produced unparseable code for names needing quotes:

  • Invalid variable and property names were exported bare instead of quoted.
  • 0x7f was treated as a valid name character, though it hasn't been allowed in unquoted identifiers since PHP 7.1.

They are in one PR because the 0x7f fix depends on the quoting added by the first commit. Kept as separate commits since each fixes its own bug, with its own test and NEWS entry.

@jorgsowa jorgsowa self-assigned this Nov 5, 2026
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