[maven-surefire] 02/02: a debug log printed in a condition only when necessary

[ti...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

tibordigana pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-surefire.git

commit d013902c0de20f83c6a85aa4fffbc2b6bd99af88
Author: tibordigana <ti...@apache.org>
AuthorDate: Fri Dec 20 23:24:04 2019 +0100

    a debug log printed in a condition only when necessary
---
 .../src/main/java/org/apache/maven/surefire/booter/ForkedBooter.java | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/surefire-booter/src/main/java/org/apache/maven/surefire/booter/ForkedBooter.java b/surefire-booter/src/main/java/org/apache/maven/surefire/booter/ForkedBooter.java
index 9da6d9b..2701221 100644
--- a/surefire-booter/src/main/java/org/apache/maven/surefire/booter/ForkedBooter.java
+++ b/surefire-booter/src/main/java/org/apache/maven/surefire/booter/ForkedBooter.java
@@ -198,15 +198,12 @@ public final class ForkedBooter
         AtomicBoolean pingDone = new AtomicBoolean( true );
         commandReader.addNoopListener( createPingHandler( pingDone ) );
         PingScheduler pingMechanisms = new PingScheduler( createPingScheduler(), ppidChecker );
-        if ( ppidChecker != null )
-        {
-            logger.debug( ppidChecker.toString() );
-        }
 
         ProcessCheckerType checkerType = startupConfiguration.getProcessChecker();
 
         if ( ( checkerType == ALL || checkerType == NATIVE ) && pingMechanisms.pluginProcessChecker != null )
         {
+            logger.debug( pingMechanisms.pluginProcessChecker.toString() );
             Runnable checkerJob = processCheckerJob( pingMechanisms );
             pingMechanisms.pingScheduler.scheduleWithFixedDelay( checkerJob, 0L, 1L, SECONDS );
         }