Thanks to visit codestin.com
Credit goes to github.com

Skip to content

I think using array better than call method_exists function #8

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
Closed
wants to merge 2 commits into from
Closed

I think using array better than call method_exists function #8

wants to merge 2 commits into from

Conversation

imanalopher
Copy link

Objects are up to twice slower at init and take up to double memory... https://gist.github.com/Thinkscape/1136563

@nicolas-grekas
Copy link
Member

Thank you @imanalopher
The benchmark you linked dates from 7 years ago. Things significantly changed since that time.
Please note also that all the work happens on the symfony/symfony repository, so this is where you should submit PRs.
If you want to submit this one, please provide recent benchmarks with and without the patch.
Please also revert all code-style related changes, since they are unrelated to the core of your proposal.
Thanks !

@imanalopher
Copy link
Author

imanalopher commented Feb 2, 2018 •
edited
Loading

Thank you @nicolas-grekas, I will back...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@imanalopher @nicolas-grekas