Move internal implementation from Zend\Code to PHP-Parser
Moving to PHP-Parser will allow inlining of parent class code, therefore reducing linear overhead and allowing more performance optimizations.
Clearing milestone: this won't happen for a while, since PHP-Parser is far from our needs right now
:+1: omg, yes - please! Need help? ;-)
@robfrawley fell free to send PRs :smile:
I'm not sure if this is interesting/viable anymore.
There is no clear advantage in using PHP-Parser unless very obvious automated optimizations can be applied.
Note: can easily be fixed if Roave/BetterReflection#39 is carefully checked
I also working on parser-reflection, working concept is ready. Main feature: all classes will extend original ones, so all existing libraries can work with them. Initialization will be compatible too: new ParserReflection\ReflectionClass(SomeClass::class)
PS: discovered a bug in the PHP-Parser during the reflection implementation testing https://github.com/nikic/PHP-Parser/pull/209 )
Yeap, BetterReflection does that as well, but @asgrim is pretty much working on it full time, so I suggest joining him on it ;-)
Marco Pivetta
http://twitter.com/Ocramius
http://ocramius.github.com/
On 6 July 2015 at 11:14, Alexander Lisachenko [email protected] wrote:
I also working on parser-reflection, working concept is ready. Main feature: all classes will extend original ones, so all existing libraries can work with them. Initialization will be compatible too: new ParserReflection\ReflectionClass(SomeClass::class)
— Reply to this email directly or view it on GitHub https://github.com/Ocramius/ProxyManager/issues/49#issuecomment-118797581 .
Yeap, BetterReflection does that as well, but @asgrim is pretty much
No, according to the https://github.com/Roave/BetterReflection/blob/master/src/Reflection/ReflectionClass.php#L11, his classes don't extend original ones, just implements an interface which looks like reflection, so I can't accept this solution for my needs. I need to reflect code transparently, like traditional reflection classes do.
@lisachenko I'm planning on creating an adapter to make this possible, see Roave/BetterReflection#18