Skip to content

ext/zend: fix canonical case of magic-method name macros#22275

Open
jorgsowa wants to merge 1 commit into
php:masterfrom
jorgsowa:debuginfo-canonical-spelling
Open

ext/zend: fix canonical case of magic-method name macros#22275
jorgsowa wants to merge 1 commit into
php:masterfrom
jorgsowa:debuginfo-canonical-spelling

Conversation

@jorgsowa

Copy link
Copy Markdown
Contributor

Extracted from #22260

Fixed the canonical case of the magic method __debugInfo() + organized all macros of magic methods with the separation of canonical and lowercase versions.

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

This makes sense to me.

@TimWolla TimWolla requested a review from Girgias June 11, 2026 20:02
@jorgsowa jorgsowa force-pushed the debuginfo-canonical-spelling branch from 24da02f to d8f2306 Compare June 11, 2026 20:31
@jorgsowa

Copy link
Copy Markdown
Contributor Author

Workflow WINDOWS_X64_ZTS failure seems to be unrelated, so I rebased the branch.

@DanielEScherzer

Copy link
Copy Markdown
Member

What is "ext/zend" in the PR title? There isn't such an extension?

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.

3 participants