Skip to content

zend_ast: Fix AST export of readonly anonymous classes - #25060

Closed
jorgsowa wants to merge 1 commit into
php:PHP-8.4from
jorgsowa:fix-ast-export-readonly-anon-class
Closed

zend_ast: Fix AST export of readonly anonymous classes#25060
jorgsowa wants to merge 1 commit into
php:PHP-8.4from
jorgsowa:fix-ast-export-readonly-anon-class

Conversation

@jorgsowa

@jorgsowa jorgsowa commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

zend_ast_export() dropped the readonly modifier for anonymous classes, so assert(new readonly class {}) showed up as new class {} in assertion messages.

@jorgsowa jorgsowa self-assigned this Oct 30, 2026
@jorgsowa jorgsowa changed the title Fix AST export of readonly anonymous classes zend_ast: Fix AST export of readonly anonymous classes Oct 30, 2026
@jorgsowa jorgsowa closed this in 5300734 Oct 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ndossche ndossche ndossche approved these changes

Assignees

@jorgsowa jorgsowa

Labels

Category: Engine

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jorgsowa @ndossche