--- jsr166/src/jsr166y/Phaser.java	2009/08/19 23:05:32	1.34
+++ jsr166/src/jsr166y/Phaser.java	2009/08/23 20:12:24	1.36
@@ -19,18 +19,22 @@ import java.util.concurrent.locks.LockSu
  *
  * <ul>
  *
- * <li> The number of parties synchronizing on a phaser may vary over
- * time.  A task may register to be a party at any time, and may
- * deregister upon arriving at the barrier.  As is the case with most
- * basic synchronization constructs, registration and deregistration
- * affect only internal counts; they do not establish any further
- * internal bookkeeping, so tasks cannot query whether they are
+ * <li> The number of parties <em>registered</em> to synchronize on a
+ * phaser may vary over time.  Tasks may be registered at any time
+ * (using methods {@link #register}, {@link #bulkRegister}, or forms
+ * of constructors establishing initial numbers of parties), and may
+ * optionally be deregistered upon any arrival (using {@link
+ * #arriveAndDeregister}).  As is the case with most basic
+ * synchronization constructs, registration and deregistration affect
+ * only internal counts; they do not establish any further internal
+ * bookkeeping, so tasks cannot query whether they are
  * registered. (However, you can introduce such bookkeeping by
  * subclassing this class.)
  *
- * <li> Each generation has an associated phase value, starting at
- * zero, and advancing when all parties reach the barrier (wrapping
- * around to zero after reaching {@code Integer.MAX_VALUE}).
+ * <li> Each generation has an associated phase number. The phase
+ * number starts at zero, amd advances when all parties arrive at the
+ * barrier, wrapping around to zero after reaching {@code
+ * Integer.MAX_VALUE}.
  *
  * <li> Like a {@code CyclicBarrier}, a phaser may be repeatedly
  * awaited.  Method {@link #arriveAndAwaitAdvance} has effect
@@ -42,14 +46,15 @@ import java.util.concurrent.locks.LockSu
  *
  *   <li> Arriving at a barrier. Methods {@link #arrive} and
  *       {@link #arriveAndDeregister} do not block, but return
- *       the phase value current upon entry to the method.
+ *       an associated <em>arrival phase number</em>;
+ *       that is, the phase number of the barrier to which the
+ *       arrival applied.
  *
  *   <li> Awaiting others. Method {@link #awaitAdvance} requires an
- *       argument indicating the entry phase, and returns when the
- *       barrier advances to a new phase.
+ *       argument indicating an arrival phase number, and returns
+ *       when the barrier advances to a new phase.
  * </ul>
  *
- *
  * <li> Barrier actions, performed by the task triggering a phase
  * advance, are arranged by overriding method {@link #onAdvance(int,
  * int)}, which also controls termination. Overriding this method is
@@ -85,6 +90,14 @@ import java.util.concurrent.locks.LockSu
  * ForkJoinPool}, which will ensure sufficient parallelism to execute
  * tasks when others are blocked waiting for a phase to advance.
  *
+ * <li>The current state of a phaser may be monitored.  At any given
+ * moment there are {@link #getRegisteredParties}, where {@link
+ * #getArrivedParties} have arrived at the current phase ({@link
+ * #getPhase}). When the remaining {@link #getUnarrivedParties})
+ * arrive, the phase advances. Method {@link #toString} returns
+ * snapshots of these state queries in a form convenient for
+ * informal monitoring.
+ *
  * </ul>
  *
  * <p><b>Sample usages:</b>
@@ -366,7 +379,7 @@ public class Phaser {
     /**
      * Adds a new unarrived party to this phaser.
      *
-     * @return the current barrier phase number upon registration
+     * @return the arrival phase number to which this registration applied
      * @throws IllegalStateException if attempting to register more
      * than the maximum supported number of parties
      */
@@ -378,7 +391,7 @@ public class Phaser {
      * Adds the given number of new unarrived parties to this phaser.
      *
      * @param parties the number of parties required to trip barrier
-     * @return the current barrier phase number upon registration
+     * @return the arrival phase number to which this registration applied
      * @throws IllegalStateException if attempting to register more
      * than the maximum supported number of parties
      */
@@ -415,8 +428,7 @@ public class Phaser {
      * Arrives at the barrier, but does not wait for others.  (You can
      * in turn wait for others via {@link #awaitAdvance}).
      *
-     * @return the barrier phase number upon entry to this method, or a
-     * negative value if terminated
+     * @return the arrival phase number, or a negative value if terminated
      * @throws IllegalStateException if not terminated and the number
      * of unarrived parties would become negative
      */
@@ -468,8 +480,7 @@ public class Phaser {
      * zero parties, this phaser also arrives at and is deregistered
      * from its parent.
      *
-     * @return the current barrier phase number upon entry to
-     * this method, or a negative value if terminated
+     * @return the arrival phase number, or a negative value if terminated
      * @throws IllegalStateException if not terminated and the number
      * of registered or unarrived parties would become negative
      */
@@ -525,7 +536,7 @@ public class Phaser {
      * method.  If instead you need to deregister upon arrival use
      * {@code arriveAndDeregister}.
      *
-     * @return the phase on entry to this method
+     * @return the arrival phase number, or a negative number if terminated
      * @throws IllegalStateException if not terminated and the number
      * of unarrived parties would become negative
      */
@@ -539,10 +550,11 @@ public class Phaser {
      * barrier is not equal to the given phase value or this barrier
      * is terminated.
      *
-     * @param phase the phase on entry to this method
-     * @return the current barrier phase number upon exit of
-     * this method, or a negative value if terminated or
-     * argument is negative
+     * @param phase an arrival phase number, or negative value if
+     * terminated; this argument is normally the value returned by a
+     * previous call to {@code arrive} or its variants
+     * @return the next arrival phase number, or a negative value
+     * if terminated or argument is negative
      */
     public int awaitAdvance(int phase) {
         if (phase < 0)
@@ -564,10 +576,11 @@ public class Phaser {
      * barrier is not equal to the given phase value or this barrier
      * is terminated.
      *
-     * @param phase the phase on entry to this method
-     * @return the current barrier phase number upon exit of
-     * this method, or a negative value if terminated or
-     * argument is negative
+     * @param phase an arrival phase number, or negative value if
+     * terminated; this argument is normally the value returned by a
+     * previous call to {@code arrive} or its variants
+     * @return the next arrival phase number, or a negative value
+     * if terminated or argument is negative
      * @throws InterruptedException if thread interrupted while waiting
      */
     public int awaitAdvanceInterruptibly(int phase)
@@ -590,14 +603,15 @@ public class Phaser {
      * returning immediately if the current phase of the barrier is not
      * equal to the given phase value or this barrier is terminated.
      *
-     * @param phase the phase on entry to this method
+     * @param phase an arrival phase number, or negative value if
+     * terminated; this argument is normally the value returned by a
+     * previous call to {@code arrive} or its variants
      * @param timeout how long to wait before giving up, in units of
      *        {@code unit}
      * @param unit a {@code TimeUnit} determining how to interpret the
      *        {@code timeout} parameter
-     * @return the current barrier phase number upon exit of
-     * this method, or a negative value if terminated or
-     * argument is negative
+     * @return the next arrival phase number, or a negative value
+     * if terminated or argument is negative
      * @throws InterruptedException if thread interrupted while waiting
      * @throws TimeoutException if timed out while waiting
      */
@@ -660,8 +674,8 @@ public class Phaser {
     }
 
     /**
-     * Returns the number of parties that have arrived at the current
-     * phase of this barrier.
+     * Returns the number of registered parties that have arrived at
+     * the current phase of this barrier.
      *
      * @return the number of arrived parties
      */