Skip to content
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

Jump to bottom

Can not cache transformed source files in the filesystem #4

Closed
lisachenko opened this issue Nov 27, 2012 · 0 comments
Closed

Can not cache transformed source files in the filesystem #4

lisachenko opened this issue Nov 27, 2012 · 0 comments
Labels
Performance

Comments

@lisachenko
Copy link
Member

Currently, only on-fly weaving is performing. This reduce overall performance as static reflection will take the place on each request. Additionally, op-code accelerators can not cache dynamic includes and just simply ignore that classes.

However, it's not easy to put transformed classes into another directory, as this will break an existing application, because DIR, FILE constants will be inside cache directory instead of original application one. ReflectionClass->getFileName() is also become incorrect.

Constants can be easily resolved during the source transforming process.
lisachenko added a commit that referenced this issue Nov 27, 2012
@lisachenko
#4 transform the source code and replace magic constants in it with r…
0f72e6b 
…eal values to work correctly from another folder.

Additionally, wrap all getFileName() methods with custom code to lead an application to believe that class is original
@gambledor gambledor mentioned this issue Dec 11, 2015
Closed
genkiroid pushed a commit to genkiroid/framework that referenced this issue Jun 23, 2016
@DavertMik
Merge pull request goaop#4 from zvirusz/patch-1
688c1dc 
Fix small typo in readme (composer.yml => composer.json)
@smuggli smuggli mentioned this issue May 15, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Performance
Milestone
No milestone
Development

No branches or pull requests
1 participant
@lisachenko