Change bootstrapper to use bulk setBeans

git-svn-id: http://anonsvn.jboss.org/repos/weld/ri/trunk@306 1c488680-804c-0410-94cd-c6b725194a0e
weld · Nov 14, 2008 · 8500b13 · 8500b13
1 parent e940985
commit 8500b13
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/webbeans-ri/src/main/java/org/jboss/webbeans/ManagerImpl.java b/webbeans-ri/src/main/java/org/jboss/webbeans/ManagerImpl.java
@@ -29,6 +29,7 @@
 import javax.webbeans.manager.Interceptor;
 import javax.webbeans.manager.Manager;
 
+import org.jboss.webbeans.bean.AbstractBean;
 import org.jboss.webbeans.bean.SimpleBean;
 import org.jboss.webbeans.bean.proxy.ProxyPool;
 import org.jboss.webbeans.contexts.ApplicationContext;
@@ -215,10 +216,10 @@ public ResolutionManager getResolutionManager()
     * Since this overwrites any existing list of beans in the manager,
     * this should only be done on startup and other controlled situations.
     * 
-    * @param beans The collection of beans to wrap.
     */
-   public Manager setBeans(Collection<Bean<?>> beans) {
+   public Manager setBeans(Set<AbstractBean<?, ?>> beans) {
       this.beans = new CopyOnWriteArrayList<Bean<?>>(beans);
+      getResolutionManager().clear();
       return this;
    }
 

diff --git a/webbeans-ri/src/main/java/org/jboss/webbeans/bootstrap/Bootstrap.java b/webbeans-ri/src/main/java/org/jboss/webbeans/bootstrap/Bootstrap.java
@@ -95,10 +95,7 @@ public void registerBeans(Class<?>...classes)
    public void registerBeans(Set<Class<?>> classes)
    {
       Set<AbstractBean<?, ?>> beans = createBeans(classes);
-      for (Bean<?> bean : beans)
-      {
-         manager.addBean(bean);
-      }
+      manager.setBeans(beans);
    }
 
    /**