Comments.

Author: jglick

src/main/java/org/jenkinsci/plugins/workflow/log/DelayBufferedOutputStream.java

@@ -116,9 +116,9 @@ private static final class Flush implements Runnable {
 
     }
 
-    //
     /**
      * Flushes streams prior to garbage collection.
+     * ({@link BufferedOutputStream} does not do this automatically.)
      * TODO Java 9+ could use java.util.Cleaner
      */
     private static final class FlushRef extends PhantomReference<DelayBufferedOutputStream> {
@@ -134,7 +134,6 @@ private static final class FlushRef extends PhantomReference<DelayBufferedOutput
                     }
                     LOGGER.log(Level.FINE, "cleaning up phantom {0}", ref.out);
                     try {
-                        // Odd that this is not the default behavior for BufferedOutputStream.
                         ref.out.flush();
                     } catch (IOException x) {
                         LOGGER.log(Level.WARNING, null, x);

src/test/java/org/jenkinsci/plugins/workflow/log/LogStorageTestBase.java

@@ -189,6 +189,7 @@ private static final class RemotePrint extends MasterToSlaveCallable<Void, Excep
             return null;
         }
     }
+    /** Checking behavior of {@link DelayBufferedOutputStream} garbage collection. */
     private static final class GC extends MasterToSlaveCallable<Void, Exception> {
         @Override public Void call() throws Exception {
             System.gc();