search

schmittjoh / JMSDiExtraBundle

Provides Advanced Dependency Injection Features for Symfony2
http://jmsyst.com/bundles/JMSDiExtraBundle
330 stars 130 forks source link

[suggestion] Moving the LazyServiceMap and Sequence to a different package #94

Open stof opened 11 years ago

stof commented 11 years ago

Currently, JMSSerializerBundle depends on this bundle only to use these lazy collections and the associated compiler passes registering them. But this implies that it inherits a dependency on JMSAopBundle and CG. And any other bundle wanting to reuse them would face the same.

what do you think about moving them to a separate package (which would not even need to be a bundle btw) ?

lsmith77 commented 11 years ago

another option is just to duplicate the code but the dependency is just way too heavy for the amount of code actually used.