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

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

Author: kabir.khan at jboss.com
Date: 2010-02-19 07:19:32 -0500 (Fri, 19 Feb 2010)
New Revision: 101159

Modified:
   projects/kernel/trunk/dependency/src/main/java/org/jboss/dependency/plugins/AbstractController.java
Log:
[JBKERNEL-95] Unnecessary read locks when accessing ConcurrentHashMap

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	2010-02-19 12:14:57 UTC (rev 101158)
+++ projects/kernel/trunk/dependency/src/main/java/org/jboss/dependency/plugins/AbstractController.java	2010-02-19 12:19:32 UTC (rev 101159)
@@ -1833,17 +1833,9 @@
     */
    protected Set<CallbackItem<?>> getCallbacks(Object name, boolean isInstallPhase)
    {
-      lockRead();
-      try
-      {
-         Map<Object, Set<CallbackItem<?>>> map = (isInstallPhase ? installCallbacks : uninstallCallbacks);
-         Set<CallbackItem<?>> callbacks = map.get(name);
-         return callbacks != null ? callbacks : Collections.<CallbackItem<?>>emptySet();
-      }
-      finally
-      {
-         unlockRead();
-      }
+      Map<Object, Set<CallbackItem<?>>> map = (isInstallPhase ? installCallbacks : uninstallCallbacks);
+      Set<CallbackItem<?>> callbacks = map.get(name);
+      return callbacks != null ? callbacks : Collections.<CallbackItem<?>>emptySet();
    }
 
    /**