--- jsr166/src/jsr166e/ConcurrentHashMapV8.java 2012/07/04 20:10:00 1.42 +++ jsr166/src/jsr166e/ConcurrentHashMapV8.java 2012/07/06 21:37:40 1.47 @@ -105,7 +105,7 @@ public class ConcurrentHashMapV8 */ public static interface MappingFunction { /** - * Returns a value for the given key, or null if there is no mapping + * Returns a value for the given key, or null if there is no mapping. * * @param key the (non-null) key * @return a value for the key, or null if none @@ -145,8 +145,8 @@ public class ConcurrentHashMapV8 * framework. As illustrated here, Spliterators are well suited to * designs in which a task repeatedly splits off half its work * into forked subtasks until small enough to process directly, - * and then joins these subtasks. Variants of this style can be - * also be used in completion-based designs. + * and then joins these subtasks. Variants of this style can also + * be used in completion-based designs. * *
      * {@code ConcurrentHashMapV8 m = ...
@@ -192,21 +192,9 @@ public class ConcurrentHashMapV8
          * @return a Spliterator covering approximately half of the
          * elements
          * @throws IllegalStateException if this Spliterator has
-         * already commenced traversing elements.
+         * already commenced traversing elements
          */
         Spliterator split();
-
-        /**
-         * Returns a Spliterator producing the same elements as this
-         * Spliterator. This method may be used for example to create
-         * a second Spliterator before a traversal, in order to later
-         * perform a second traversal.
-         *
-         * @return a Spliterator covering the same range as this Spliterator.
-         * @throws IllegalStateException if this Spliterator has
-         * already commenced traversing elements.
-         */
-        Spliterator clone();
     }
 
     /*
@@ -806,7 +794,7 @@ public class ConcurrentHashMapV8
         }
 
         /**
-         * Find or add a node
+         * Finds or adds a node.
          * @return null if added
          */
         @SuppressWarnings("unchecked") // suppress Comparable cast warning
@@ -2012,8 +2000,8 @@ public class ConcurrentHashMapV8
     }
 
     /**
-     * Split a normal bin with list headed by e into lo and hi parts;
-     * install in given table
+     * Splits a normal bin with list headed by e into lo and hi parts;
+     * installs in given table.
      */
     private static void splitBin(Node[] nextTab, int i, Node e) {
         int bit = nextTab.length >>> 1; // bit to split on
@@ -2043,7 +2031,7 @@ public class ConcurrentHashMapV8
     }
 
     /**
-     * Split a tree bin into lo and hi parts; install in given table
+     * Splits a tree bin into lo and hi parts; installs in given table.
      */
     private static void splitTreeBin(Node[] nextTab, int i, TreeBin t) {
         int bit = nextTab.length >>> 1;
@@ -2199,7 +2187,7 @@ public class ConcurrentHashMapV8
             baseLimit = baseSize = (tab == null) ? 0 : tab.length;
         }
 
-        /** Creates iterator for clone() and split() methods */
+        /** Creates iterator for clone() and split() methods. */
         InternalIterator(InternalIterator it, boolean split) {
             this.map = it.map;
             this.tab = it.tab;