You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ti...@apache.org on 2015/11/28 08:43:22 UTC

[2/3] maven-surefire git commit: refactoring

refactoring


Project: http://git-wip-us.apache.org/repos/asf/maven-surefire/repo
Commit: http://git-wip-us.apache.org/repos/asf/maven-surefire/commit/47bf4b85
Tree: http://git-wip-us.apache.org/repos/asf/maven-surefire/tree/47bf4b85
Diff: http://git-wip-us.apache.org/repos/asf/maven-surefire/diff/47bf4b85

Branch: refs/heads/master
Commit: 47bf4b851093c6c785c14f1e68e6c14decf60f62
Parents: c643b32
Author: Tibor17 <ti...@lycos.com>
Authored: Sun Nov 22 23:17:47 2015 +0100
Committer: Tibor17 <ti...@lycos.com>
Committed: Sun Nov 22 23:17:47 2015 +0100

----------------------------------------------------------------------
 .../maven/surefire/booter/MasterProcessReader.java    | 14 ++++++++++++++
 1 file changed, 14 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/47bf4b85/surefire-api/src/main/java/org/apache/maven/surefire/booter/MasterProcessReader.java
----------------------------------------------------------------------
diff --git a/surefire-api/src/main/java/org/apache/maven/surefire/booter/MasterProcessReader.java b/surefire-api/src/main/java/org/apache/maven/surefire/booter/MasterProcessReader.java
index a114154..dabe80a 100644
--- a/surefire-api/src/main/java/org/apache/maven/surefire/booter/MasterProcessReader.java
+++ b/surefire-api/src/main/java/org/apache/maven/surefire/booter/MasterProcessReader.java
@@ -353,6 +353,20 @@ public final class MasterProcessReader
                 do
                 {
                     requestNextTest();
+                    /**
+                     * this branching should be refactored to
+                     * waitNextTest();
+                     * if ( isStopped() )
+                     * {
+                     *      clazz = null;
+                     *      return;
+                     * }
+                     * clazz = current.item;
+                     * if ( !isLastNode( current ) )
+                     * {
+                     *      current = current.successor.get();
+                     * }
+                     */
                     if ( isLastNode( current ) )
                     {
                         clazz = null;