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

Container compiler #133

Closed
wants to merge 21 commits into from
Closed

Container compiler #133

wants to merge 21 commits into from

Conversation

mnapoli
Copy link
Member

@mnapoli mnapoli commented Nov 24, 2013

Compile the container to PHP code for best performances.

TODO:

  • ContainerCompiler
  • ValueDefinitionCompiler
  • AliasDefinitionCompiler
  • ClassDefinitionCompiler
  • CallbackDefinitionCompiler
  • Lazy reading/loading of definition sources (read definitions lazily bc the compiled container doesn't need them)
  • Work on performances: mnapoli/di-benchmark
  • Compile all known definitions upstart in a single file

I did mess some things up with git, I should have put some commits in the 4.0 branch instead of this one :/

@ghost ghost assigned mnapoli Nov 24, 2013
@mnapoli
PSR-2 love
124a0b7
@mnapoli
Updated the project URL
b9f9652
@mnapoli
ValueDefinition compiler
26792c4
@mnapoli
ClassDefinition compiler
709eb53
@mnapoli
AliasDefinition compiler
8d2b083
@mnapoli
CallableDefinition compiler
cc2e254
@mnapoli
Compiler backend: FileBackend (stores to PHP files and include them)
c564e4c
@mnapoli
Compiler
2c705ab
@mnapoli
Code formatting
1cbf680
@mnapoli
Added integration test for scope
943c106
@mnapoli
First version of CompiledContainer
fc30486
@mnapoli
Integration tests for compiled container
7ed71d0
@mnapoli
Added factory (callable) integration tests
17c016a
@mnapoli
Fixed tests after the merge
37624c0
@mnapoli
Merge branch '4.0' into feature/Compiler
19a96a8 
Conflicts:
	src/DI/Container.php
	src/DI/Definition/Resolver/FactoryDefinitionResolver.php
	tests/IntegrationTests/DI/InjectionTest.php
	tests/UnitTests/DI/ContainerBuilderTest.php
@mnapoli
Fixed everything after big merge
71ebd55
@mnapoli
Renamed CallableDefinitionCompiler to FactoryDefinitionCompiler
ec8bc1b
@mnapoli
Moved DefinitionCompilers to the Definition\Compiler namespace
8476e8d
@mnapoli
Optimization in CompiledContainer::has()
8c72419
@mnapoli
Merge branch '4.0' into feature/Compiler
527894b
@mnapoli
#146 Tests and fix for injecting null values in parameters and proper…
f69459c 
…ties
@mnapoli
Copy link
Member Author

mnapoli commented Jan 25, 2014

Closing in favor of #149 (pull request based on master)

@mnapoli mnapoli closed this Jan 25, 2014
@mnapoli mnapoli mentioned this pull request May 28, 2017
Merged
31 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mnapoli mnapoli

Labels
enhancement
Projects
None yet
Milestone
4.1
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@mnapoli