Allow disabled logging when JAVA_HOME is not known (#52)

jwgmeligmeyling · olamy · commit e35578547943 · 2018-07-15T10:38:46.000+02:00
diff --git a/plexus-compilers/plexus-compiler-javac/src/main/java/org/codehaus/plexus/compiler/javac/JavacCompiler.java b/plexus-compilers/plexus-compiler-javac/src/main/java/org/codehaus/plexus/compiler/javac/JavacCompiler.java
@@ -158,7 +158,9 @@ public CompilerResult performCompile( CompilerConfiguration config )
                 }
                 catch ( IOException e )
                 {
-                    getLogger().warn( "Unable to autodetect 'javac' path, using 'javac' from the environment." );
+                    if ( (getLogger() != null ) && getLogger().isWarnEnabled()) {
+                        getLogger().warn( "Unable to autodetect 'javac' path, using 'javac' from the environment." );
+                    }
                     executable = "javac";
                 }
             }
@@ -590,7 +592,9 @@ CompilerResult compileInProcess( String[] args, CompilerConfiguration config )
         final Thread thread = Thread.currentThread();
         final ClassLoader contextClassLoader = thread.getContextClassLoader();
         thread.setContextClassLoader( javacClass.getClassLoader() );
-        getLogger().debug( "ttcl changed run compileInProcessWithProperClassloader" );
+        if ( (getLogger() != null ) && getLogger().isDebugEnabled()) {
+            getLogger().debug("ttcl changed run compileInProcessWithProperClassloader");
+        }
         try
         {
             return compileInProcessWithProperClassloader(javacClass, args);

Original file line number	Diff line number	Diff line change
`@@ -158,7 +158,9 @@ public CompilerResult performCompile( CompilerConfiguration config )`
`158`	`158`	`}`
`159`	`159`	`catch ( IOException e )`
`160`	`160`	`{`
`161`		`- getLogger().warn( "Unable to autodetect 'javac' path, using 'javac' from the environment." );`
	`161`	`+ if ( (getLogger() != null ) && getLogger().isWarnEnabled()) {`
	`162`	`+ getLogger().warn( "Unable to autodetect 'javac' path, using 'javac' from the environment." );`
	`163`	`+ }`
`162`	`164`	`executable = "javac";`
`163`	`165`	`}`
`164`	`166`	`}`
`@@ -590,7 +592,9 @@ CompilerResult compileInProcess( String[] args, CompilerConfiguration config )`
`590`	`592`	`final Thread thread = Thread.currentThread();`
`591`	`593`	`final ClassLoader contextClassLoader = thread.getContextClassLoader();`
`592`	`594`	`thread.setContextClassLoader( javacClass.getClassLoader() );`
`593`		`- getLogger().debug( "ttcl changed run compileInProcessWithProperClassloader" );`
	`595`	`+ if ( (getLogger() != null ) && getLogger().isDebugEnabled()) {`
	`596`	`+ getLogger().debug("ttcl changed run compileInProcessWithProperClassloader");`
	`597`	`+ }`
`594`	`598`	`try`
`595`	`599`	`{`
`596`	`600`	`return compileInProcessWithProperClassloader(javacClass, args);`