Addresses PR review comments for #233

Author: viktorklang

tck/src/test/java/org/reactivestreams/tck/SyncTriggeredDemandSubscriberTest.java

@@ -37,7 +37,6 @@ public SyncTriggeredDemandSubscriberTest() {
       }
 
       @Override public void onComplete() {
-        System.out.println("Accumulated: " + acc);
       }
     };
   }

tck/src/test/java/org/reactivestreams/tck/support/SyncTriggeredDemandSubscriber.java

@@ -4,9 +4,10 @@
 import org.reactivestreams.Subscription;
 
 /**
- * SyncSubscriber is an implementation of Reactive Streams `Subscriber`,
- * it runs synchronously (on the Publisher's thread) and requests one element
- * at a time and invokes a user-defined method to process each element.
+ * SyncTriggeredDemandSubscriber is an implementation of Reactive Streams `Subscriber`,
+ * it runs synchronously (on the Publisher's thread) and requests demand triggered from
+ * "the outside" using its `triggerDemand` method and from "the inside" using the return
+ * value of its user-defined `foreach` method which is invoked to process each element.
  *
  * NOTE: The code below uses a lot of try-catches to show the reader where exceptions can be expected, and where they are forbidden.
  */
@@ -32,6 +33,11 @@ public abstract class SyncTriggeredDemandSubscriber<T> implements Subscriber<T>
     }
   }
 
+  /**
+   * Requests the provided number of elements from the `Subscription` of this `Subscriber`.
+   * NOTE: This makes no attempt at thread safety so only invoke it once from the outside to initiate the demand.
+   * @return `true` if successful and `false` if not (either due to no `Subscription` or due to exceptions thrown)
+   */
   public boolean triggerDemand(final long n) {
     final Subscription s = subscription;
     if (s == null) return false;