Skip to content

reflection: pass zend_object* to _class_string() - #23279

Open
Girgias wants to merge 1 commit into
php:masterfrom
Girgias:2026-08-reflection-minor-refactoring
Open

reflection: pass zend_object* to _class_string()#23279
Girgias wants to merge 1 commit into
php:masterfrom
Girgias:2026-08-reflection-minor-refactoring

Conversation

@Girgias

@Girgias Girgias commented Aug 14, 2026

Copy link
Copy Markdown
Member

Rather than a zval

@Girgias

Girgias commented Aug 14, 2026

Copy link
Copy Markdown
Member Author

It might be a good idea in the future to refactor intern->obj to be a zend_object* rather than a zval. This only saves 8 bytes in the struct, but I think it would make the assumptions and logic surrounding this field easier to grasp.

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