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) {