You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by ju...@apache.org on 2013/04/26 10:36:58 UTC

svn commit: r1476101 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/TreeImpl.java

Author: jukka
Date: Fri Apr 26 08:36:58 2013
New Revision: 1476101

URL: http://svn.apache.org/r1476101
Log:
OAK-781: Clarify / fix effects of MISSING_NODE as base state of NodeBuilder

No more need to get a NodeState snapshot for the exists() call in canRead(TreeImpl)

Modified:
    jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/TreeImpl.java

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/TreeImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/TreeImpl.java?rev=1476101&r1=1476100&r2=1476101&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/TreeImpl.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/TreeImpl.java Fri Apr 26 08:36:58 2013
@@ -536,7 +536,7 @@ public class TreeImpl implements Tree {
     private boolean canRead(TreeImpl tree) {
         // TODO: OAK-753 TreeImpl exposes hidden child trees
         // return tree.getNodeState().exists() && !NodeStateUtils.isHidden(tree.getName());
-        return tree.getNodeState().exists();
+        return tree.nodeBuilder.exists();
     }
 
     private boolean canRead(PropertyState property) {