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 md...@apache.org on 2013/06/07 10:16:26 UTC
svn commit: r1490534 -
/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/RootImpl.java
Author: mduerig
Date: Fri Jun 7 08:16:25 2013
New Revision: 1490534
URL: http://svn.apache.org/r1490534
Log:
OAK-709: Consider moving permission evaluation to the node state level
convert secureBuilder field to local since it is not used referenced from elsewhere
Modified:
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/RootImpl.java
Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/RootImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/RootImpl.java?rev=1490534&r1=1490533&r2=1490534&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/RootImpl.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/RootImpl.java Fri Jun 7 08:16:25 2013
@@ -101,8 +101,10 @@ public class RootImpl implements Root {
*/
private NodeStoreBranch branch;
- private NodeBuilder rawBuilder;
- private NodeBuilder secureBuilder;
+ /**
+ * Unsecured builder for the root tree
+ */
+ private NodeBuilder builder;
/** Sentinel for the next move operation to take place on the this root */
private Move lastMove = new Move();
@@ -142,8 +144,8 @@ public class RootImpl implements Root {
branch = this.store.branch();
NodeState root = branch.getHead();
- rawBuilder = root.builder();
- secureBuilder = new SecureNodeBuilder(rawBuilder, getRootContext(root));
+ builder = root.builder();
+ NodeBuilder secureBuilder = new SecureNodeBuilder(builder, getRootContext(root));
rootTree = new TreeImpl(this, secureBuilder, lastMove);
}
@@ -403,7 +405,7 @@ public class RootImpl implements Root {
*/
@Nonnull
private NodeState getRootState() {
- return rawBuilder.getNodeState();
+ return builder.getNodeState();
}
/**
@@ -443,7 +445,7 @@ public class RootImpl implements Root {
*/
private void reset() {
NodeState root = branch.getHead();
- rawBuilder.reset(root);
+ builder.reset(root);
}
@Nonnull