Skip to content

[DI] Optimize use of private and pre-defined services #23561

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.

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

Merged
merged 1 commit into from
Jul 19, 2017

Conversation

nicolas-grekas
Copy link
Member

@nicolas-grekas nicolas-grekas commented Jul 17, 2017

Q A
Branch? master
Bug fix? no
New feature? yes (perf)
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -

By making private services really private and taking into account that (un)setting pre-defined services is not allowed anymore, we can go one step further into optimizing the dumped container.

@nicolas-grekas
Copy link
Member Author

rebased and ready

@fabpot
Copy link
Member

fabpot commented Jul 19, 2017

Thank you @nicolas-grekas.

@fabpot fabpot merged commit c0c1881 into symfony:master Jul 19, 2017
fabpot added a commit that referenced this pull request Jul 19, 2017
…(nicolas-grekas)

This PR was merged into the 4.0-dev branch.

Discussion
----------

[DI] Optimize use of private and pre-defined services

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes (perf)
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

By making private services really private and taking into account that (un)setting pre-defined services is not allowed anymore, we can go one step further into optimizing the dumped container.

Commits
-------

c0c1881 [DI] Optimize use of private and pre-defined services
@stof
Copy link
Member

stof commented Jul 24, 2017

I think you broke the reset() method, as $this->privates must be reset too in dumped containers now. You need to extend this method in dumped containers

@nicolas-grekas nicolas-grekas deleted the private branch July 24, 2017 14:01
@fabpot fabpot mentioned this pull request Oct 19, 2017
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.

4 participants