You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jackrabbit.apache.org by st...@apache.org on 2005/08/26 09:56:34 UTC

svn commit: r240184 - in /incubator/jackrabbit/trunk/core/src/java/org/apache/jackrabbit/core: CachingHierarchyManager.java HierarchyManagerImpl.java

Author: stefan
Date: Fri Aug 26 00:56:28 2005
New Revision: 240184

URL: http://svn.apache.org/viewcvs?rev=240184&view=rev
Log:
remove unnecessary 'synchronized' method modifiers to reduce dead-lock potential

Modified:
    incubator/jackrabbit/trunk/core/src/java/org/apache/jackrabbit/core/CachingHierarchyManager.java
    incubator/jackrabbit/trunk/core/src/java/org/apache/jackrabbit/core/HierarchyManagerImpl.java

Modified: incubator/jackrabbit/trunk/core/src/java/org/apache/jackrabbit/core/CachingHierarchyManager.java
URL: http://svn.apache.org/viewcvs/incubator/jackrabbit/trunk/core/src/java/org/apache/jackrabbit/core/CachingHierarchyManager.java?rev=240184&r1=240183&r2=240184&view=diff
==============================================================================
--- incubator/jackrabbit/trunk/core/src/java/org/apache/jackrabbit/core/CachingHierarchyManager.java (original)
+++ incubator/jackrabbit/trunk/core/src/java/org/apache/jackrabbit/core/CachingHierarchyManager.java Fri Aug 26 00:56:28 2005
@@ -193,7 +193,7 @@
      * Overridden method simply checks whether we have an item matching the id
      * and returns its path, otherwise calls base implementation.
      */
-    public synchronized Path getPath(ItemId id)
+    public Path getPath(ItemId id)
             throws ItemNotFoundException, RepositoryException {
 
         if (id.denotesNode()) {

Modified: incubator/jackrabbit/trunk/core/src/java/org/apache/jackrabbit/core/HierarchyManagerImpl.java
URL: http://svn.apache.org/viewcvs/incubator/jackrabbit/trunk/core/src/java/org/apache/jackrabbit/core/HierarchyManagerImpl.java?rev=240184&r1=240183&r2=240184&view=diff
==============================================================================
--- incubator/jackrabbit/trunk/core/src/java/org/apache/jackrabbit/core/HierarchyManagerImpl.java (original)
+++ incubator/jackrabbit/trunk/core/src/java/org/apache/jackrabbit/core/HierarchyManagerImpl.java Fri Aug 26 00:56:28 2005
@@ -319,7 +319,7 @@
     /**
      * {@inheritDoc}
      */
-    public synchronized ItemId resolvePath(Path path)
+    public ItemId resolvePath(Path path)
             throws PathNotFoundException, RepositoryException {
         // shortcut
         if (path.denotesRoot()) {
@@ -338,7 +338,7 @@
     /**
      * {@inheritDoc}
      */
-    public synchronized Path getPath(ItemId id)
+    public Path getPath(ItemId id)
             throws ItemNotFoundException, RepositoryException {
         // shortcut
         if (id.equals(rootNodeId)) {