[jboss-dev-forums] [Design the new POJO MicroContainer] - Re: Context installation causes StackOverflowError

[adrian@jboss.org]

Incidently, I think there's a further optimization where we don't look at the
dependencies for contexts that are already in the "installing' set, since we know
we won't process them anyway.


  |    protected Set<ControllerContext> resolveContexts(Set<ControllerContext> contexts, ControllerState state, boolean trace)
  |    {
  |       HashSet<ControllerContext> result = new HashSet<ControllerContext>();
  | 
  |       if (contexts.isEmpty() == false)
  |       {
  |          for (ControllerContext ctx : contexts)
  |          {
  | -            if (advance(ctx))
  | +            if (installing.contains(context) == false && advance(ctx))
  |             {
  | 

It might be an idea to test that for backwards compatiblity issuse at the same time?

View the original post : http://www.jboss.org/index.html?module=bb&op=viewtopic&p=4220291#4220291

Reply to the post : http://www.jboss.org/index.html?module=bb&op=posting&mode=reply&p=4220291