You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jackrabbit.apache.org by an...@apache.org on 2009/11/05 22:29:59 UTC

svn commit: r833183 - in /jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi: ItemManagerImpl.java LazyItemIterator.java NodeImpl.java RepositoryImpl.java WorkspaceImpl.java WorkspaceManager.java

Author: angela
Date: Thu Nov  5 21:29:58 2009
New Revision: 833183

URL: http://svn.apache.org/viewvc?rev=833183&view=rev
Log:
fix typos in comments and javadoc, improve readability of comments

Modified:
    jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/ItemManagerImpl.java
    jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/LazyItemIterator.java
    jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/NodeImpl.java
    jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/RepositoryImpl.java
    jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/WorkspaceImpl.java
    jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/WorkspaceManager.java

Modified: jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/ItemManagerImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/ItemManagerImpl.java?rev=833183&r1=833182&r2=833183&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/ItemManagerImpl.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/ItemManagerImpl.java Thu Nov  5 21:29:58 2009
@@ -97,7 +97,7 @@
         if (wsp instanceof WorkspaceImpl) {
             ((WorkspaceImpl) wsp).getItemStateFactory().removeCreationListener(this);
         }
-        // aftwards clear the cache.
+        // ... and clear the cache.
         itemCache.clear();
     }
 

Modified: jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/LazyItemIterator.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/LazyItemIterator.java?rev=833183&r1=833182&r2=833183&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/LazyItemIterator.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/LazyItemIterator.java Thu Nov  5 21:29:58 2009
@@ -90,7 +90,7 @@
             size = UNDEFINED_SIZE;
         }
         pos = 0;
-        // prefetch first item
+        // fetch first item
         next = prefetchNext();
     }
 
@@ -119,7 +119,7 @@
         iter = entries.iterator();
         size = entries.size();
         pos = 0;
-        // prefetch first item
+        // fetch first item
         next = prefetchNext();
     }
 
@@ -223,7 +223,7 @@
                 entry = iter.next();
             }
         }
-        // prefetch final item (the one to be returned on next())
+        // fetch final item (the one to be returned on next())
         pos++;
         next = prefetchNext();
     }

Modified: jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/NodeImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/NodeImpl.java?rev=833183&r1=833182&r2=833183&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/NodeImpl.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/NodeImpl.java Thu Nov  5 21:29:58 2009
@@ -393,7 +393,7 @@
      * @see Node#setProperty(String, Node)
      */
     public Property setProperty(String name, Node value) throws ValueFormatException, VersionException, LockException, ConstraintViolationException, RepositoryException {
-        // duplicate check to make sure, writability is asserted before value
+        // duplicate check to make sure, property can be written before value
         // validation below.
         checkIsWritable();
         Value v;
@@ -1082,7 +1082,7 @@
     public void restore(Version version, String relPath, boolean removeExisting) throws PathNotFoundException, ItemExistsException, VersionException, ConstraintViolationException, UnsupportedRepositoryOperationException, LockException, InvalidItemStateException, RepositoryException {
         checkSessionHasPendingChanges();
 
-        // additional checks are performed with subsequest calls.
+        // additional checks are performed with subsequent calls.
         if (hasNode(relPath)) {
             // node at 'relPath' exists -> call restore on the target Node
             getNode(relPath).restore(version, removeExisting);
@@ -1626,7 +1626,7 @@
 
         // get list of existing nodetypes
         Name[] existingNts = getNodeState().getNodeTypeNames();
-        // build effective node type representing primary type including existing mixin's
+        // build effective node type representing primary type including existing mixins
         EffectiveNodeType entExisting = session.getEffectiveNodeTypeProvider().getEffectiveNodeType(existingNts);
 
         // check if the base type supports adding this mixin
@@ -1771,7 +1771,7 @@
             if (rp.getLength() == 1 && rp.getNameElement().denotesName()) {
                 // a single path element must always denote a name. '.' and '..'
                 // will never point to a property. If the NodeEntry does not
-                // contain such a pe, the targetEntry is 'null;
+                // contain such a property entry, the targetEntry is 'null;
                 Name propName = rp.getNameElement().getName();
                 // check if property entry exists
                 targetEntry = getNodeEntry().getPropertyEntry(propName, true);

Modified: jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/RepositoryImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/RepositoryImpl.java?rev=833183&r1=833182&r2=833183&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/RepositoryImpl.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/RepositoryImpl.java Thu Nov  5 21:29:58 2009
@@ -161,8 +161,7 @@
         }
     }
 
-    //---------------------------------------------------------< Rereferencable >---
-
+    //------------------------------------------------------< Referenceable >---
     /**
      * @see Referenceable#getReference()
      */

Modified: jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/WorkspaceImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/WorkspaceImpl.java?rev=833183&r1=833182&r2=833183&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/WorkspaceImpl.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/WorkspaceImpl.java Thu Nov  5 21:29:58 2009
@@ -141,7 +141,7 @@
         session.checkSupportedOption(Repository.LEVEL_2_SUPPORTED);
         session.checkIsAlive();
 
-        // do intra-workspace copy
+        // do within workspace copy
         Path srcPath = session.getQPath(srcAbsPath);
         Path destPath = session.getQPath(destAbsPath);
 
@@ -158,7 +158,7 @@
 
         // check workspace name
         if (getName().equals(srcWorkspace)) {
-            // same as current workspace, delegate to intra-workspace copy method
+            // same as current workspace, delegate to within workspace copy method
             copy(srcAbsPath, destAbsPath);
             return;
         }

Modified: jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/WorkspaceManager.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/WorkspaceManager.java?rev=833183&r1=833182&r2=833183&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/WorkspaceManager.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/WorkspaceManager.java Thu Nov  5 21:29:58 2009
@@ -566,8 +566,8 @@
             /*
             Execute operation and delegate invalidation of affected item
             states to the operation.
-            NOTE, that the invalidation is independant of the cache behaviour
-            due to the fact, that local eventbundles are not processed by
+            NOTE, that the invalidation is independent of the cache behaviour
+            due to the fact, that local event bundles are not processed by
             the HierarchyEventListener.
             */
             new OperationVisitorImpl(sessionInfo).execute(operation);