[jboss-cvs] JBossAS SVN: r95997 - projects/kernel/trunk/dependency/src/main/java/org/jboss/dependency/plugins.

[jboss-cvs-commits at lists.jboss.org]

Author: alesj
Date: 2009-11-04 09:52:55 -0500 (Wed, 04 Nov 2009)
New Revision: 95997

Modified:
   projects/kernel/trunk/dependency/src/main/java/org/jboss/dependency/plugins/AbstractController.java
Log:
Fix callback tracking cache.

Modified: projects/kernel/trunk/dependency/src/main/java/org/jboss/dependency/plugins/AbstractController.java
===================================================================
--- projects/kernel/trunk/dependency/src/main/java/org/jboss/dependency/plugins/AbstractController.java	2009-11-04 14:12:18 UTC (rev 95996)
+++ projects/kernel/trunk/dependency/src/main/java/org/jboss/dependency/plugins/AbstractController.java	2009-11-04 14:52:55 UTC (rev 95997)
@@ -1620,7 +1620,7 @@
       {
          Map<Object, Set<CallbackItem<?>>> map = (isInstallPhase ? installCallbacks : uninstallCallbacks);
          Set<CallbackItem<?>> callbacks = map.get(name);
-         return callbacks != null ? callbacks : new HashSet<CallbackItem<?>>();
+         return callbacks != null ? callbacks : Collections.<CallbackItem<?>>emptySet();
       }
       finally
       {
@@ -1693,7 +1693,8 @@
          if (dependencyInfo != null && dependencyInfo.isAutowireCandidate())
          {
             // match callbacks by name
-            Set<CallbackItem<?>> existingCallbacks = getCallbacks(context.getName(), isInstallPhase);
+            Set<CallbackItem<?>> existingCallbacks = new HashSet<CallbackItem<?>>();
+            existingCallbacks.addAll(getCallbacks(context.getName(), isInstallPhase));
             // match by classes
             Collection<Class<?>> classes = getClassesImplemented(context.getTarget());
             if (classes != null && classes.isEmpty() == false)