--- jsr166/src/test/loops/BinaryAsyncAction.java 2012/11/26 11:36:33 1.13 +++ jsr166/src/test/loops/BinaryAsyncAction.java 2015/01/15 18:42:39 1.16 @@ -13,7 +13,7 @@ import java.util.concurrent.atomic.*; * have smaller stack space footprints and faster completion mechanics * but higher per-task footprints. Compared to LinkedAsyncActions, * BinaryAsyncActions are simpler to use and have less overhead in - * typical uasges but are restricted to binary computation trees. + * typical usages but are restricted to binary computation trees. * *

Upon construction, a BinaryAsyncAction does not bear any * linkages. For non-root tasks, links must be established using @@ -38,7 +38,7 @@ import java.util.concurrent.atomic.*; * } * } * - * An alternative, and usually faster strategy is to instead use a + * An alternative, and usually faster, strategy is to instead use a * loop to fork subtasks: *

  *   protected void compute() {
@@ -89,7 +89,7 @@ public abstract class BinaryAsyncAction
      * as parent, and each other as siblings.
      * @param x one subtask
      * @param y the other subtask
-     * @throws NullPointerException if either argument is null.
+     * @throws NullPointerException if either argument is null
      */
     public final void linkSubtasks(BinaryAsyncAction x, BinaryAsyncAction y) {
         x.parent = y.parent = this;
@@ -120,7 +120,7 @@ public abstract class BinaryAsyncAction
      * default version of this method does nothing and returns
      * {@code true}.
      * @return true if this task's exception should be propagated to
-     * this task's parent.
+     * this task's parent
      */
     protected boolean onException() {
         return true;
@@ -192,7 +192,7 @@ public abstract class BinaryAsyncAction
      * @param ex the exception to throw when joining this task
      * @throws NullPointerException if ex is null
      * @throws Throwable if any invocation of
-     * {@code onException} does so.
+     * {@code onException} does so
      */
     public final void completeExceptionally(Throwable ex) {
         BinaryAsyncAction a = this;
@@ -209,7 +209,7 @@ public abstract class BinaryAsyncAction
     /**
      * Returns this task's parent, or null if none or this task
      * is already complete.
-     * @return this task's parent, or null if none.
+     * @return this task's parent, or null if none
      */
     public final BinaryAsyncAction getParent() {
         return parent;
@@ -218,7 +218,7 @@ public abstract class BinaryAsyncAction
     /**
      * Returns this task's sibling, or null if none or this task is
      * already complete.
-     * @return this task's sibling, or null if none.
+     * @return this task's sibling, or null if none
      */
     public BinaryAsyncAction getSibling() {
         return sibling;
@@ -267,7 +267,7 @@ public abstract class BinaryAsyncAction
     }
 
     /**
-     * Sets the control state to the given value,
+     * Increments the control state.
      * @param value the new value
      */
     protected final void incrementControlState() {
@@ -275,7 +275,7 @@ public abstract class BinaryAsyncAction
     }
 
     /**
-     * Decrement the control state
+     * Decrements the control state.
      * @return true if successful
      */
     protected final void decrementControlState() {