Skip to content

ext/reflection: various cleanups and simplifications #22564

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
DanielEScherzer wants to merge 33 commits into
base: master
Choose a base branch
from
+240 −195
Open

ext/reflection: various cleanups and simplifications #22564

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
33 commits
Select commit Hold shift + click to select a range
310552a
Implement `ReflectionClass::isAnonymous()` using `_class_check_flag()`
DanielEScherzer Jul 2, 2026
dbb4746
Reflection: use `RETURN_BOOL` in `_class_check_flag()`
DanielEScherzer Jul 3, 2026
c6a0bd2
Implement `ReflectionClassConstant::isEnumCase()` using existing helper
DanielEScherzer Jul 2, 2026
c387624
Implement `ReflectionClassConstant::isDeprecated()` using existing he…
DanielEScherzer Jul 2, 2026
456b7b7
Fix inline documentation of `ReflectionProperty::getName()`
DanielEScherzer Jul 2, 2026
5f03270
Update inline documentation of `ReflectionProperty::setAccessible()`
DanielEScherzer Jul 2, 2026
89e0ada
Fix inline documentation of `Reflection*::getModifiers()` methods
DanielEScherzer Jul 2, 2026
6e1d9bc
Fix inline documentation of `ReflectionType::allowsNull()`
DanielEScherzer Jul 2, 2026
1dd00fe
Fix inline documentation of `ReflectionMethod::getClosure()`
DanielEScherzer Jul 2, 2026
4f905f7
Avoid unnecessary concatenation in `zend_named_reflection_type_to_str…
DanielEScherzer Jul 2, 2026
6db4eee
Reflection: use `true` and `false` for booleans rather than `0`/`1`
DanielEScherzer Jul 2, 2026
c7bb898
Reflection: convert some integers used for conditions to booleans
DanielEScherzer Jul 2, 2026
24d6833
Reflection: remove unneeded indentation support from `_extension_stri…
DanielEScherzer Jul 2, 2026
00db5d3
Reflection: remove unneeded indentation support from `_extension_ini_…
DanielEScherzer Jul 2, 2026
fc43900
Reflection: remove unneeded indentation support from `_zend_extension…
DanielEScherzer Jul 2, 2026
8399a44
Reflection: access common function fields through `zend_function.common`
DanielEScherzer Jul 2, 2026
ac61d97
`ReflectionNamedType::getName()`: inline logic for legacy behavior
DanielEScherzer Jul 2, 2026
b133e1f
`ReflectionFunction::__toString()`: stop accessing `intern->ce`
DanielEScherzer Jul 2, 2026
be1d337
`ReflectionFunctionAbstract::getClosureUsedVariables()`: optimize arrays
DanielEScherzer Jul 2, 2026
f5813be
`ReflectionClass::getStaticProperties()`: optimize array initialization
DanielEScherzer Jul 2, 2026
48c6e8e
Fix inline documentation of tentative return methods
DanielEScherzer Jul 2, 2026
8d63d59
`ReflectionFunctionAbstract::getClosureCalledClass()`: remove pointer…
DanielEScherzer Jul 2, 2026
2335a18
Reflection: remove `reflection_extension_factory()`, avoid extra lookups
DanielEScherzer Jul 2, 2026
d533f87
Reflection: rename `reflection_extension_factory_ex()`
DanielEScherzer Jul 2, 2026
841c9ad
Reflection: improve string release in `reflection_property_factory_st…
DanielEScherzer Jul 2, 2026
2ad966f
Simplify `ReflectionProperty::getSettableType()` property handling
DanielEScherzer Jul 2, 2026
230f4fa
Rename `ReflectionParameter_isDefault.phpt` to reflect actual contents
DanielEScherzer Jul 2, 2026
4bec8ca
Update `ReflectionMethod_getClosureThis.phpt` to match actual contents
DanielEScherzer Jul 2, 2026
69c2716
Fix inline comment in `ReflectionMethod::isConstructor()`
DanielEScherzer Jul 2, 2026
23bfb22
Reflection: minor whitespace cleanup
DanielEScherzer Jul 2, 2026
316332d
`ReflectionAttribute::getArguments()`: optimize array initialization
DanielEScherzer Jul 2, 2026
7c134cd
`ReflectionClass::get(Reflection)Constants()`: optimize array initial…
DanielEScherzer Jul 2, 2026
e3a7f4b
`ReflectionClass::isSubclassOf()`: remove extraneous parentheses
DanielEScherzer Jul 2, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Loading
Loading