fix methods of lint and warning see #131 (comment) (#132)

Signed-off-by: olivier lamy <olamy@apache.org>

Author: olamy

plexus-compiler-api/src/main/java/org/codehaus/plexus/compiler/CompilerConfiguration.java

@@ -70,6 +70,8 @@ public class CompilerConfiguration
     private boolean showWarnings = true;
 
     private String warnings;
+    
+    private boolean showLint;
 
     /**
      * -Werror argument as supported since Java 1.7
@@ -341,16 +343,21 @@ public String getDebugLevel()
         return debugLevel;
     }
 
-    public void setShowWarnings( boolean showWarnings )
+    public void setWarnings( String warnings )
     {
-        this.showWarnings = showWarnings;
+        this.warnings = warnings;
     }
 
     public boolean isShowWarnings()
     {
         return showWarnings;
     }
 
+    public void setShowWarnings( boolean showWarnings )
+    {
+        this.showWarnings = showWarnings;
+    }
+
     public boolean isShowDeprecation()
     {
         return showDeprecation;
@@ -361,9 +368,16 @@ public String getWarnings()
         return warnings;
     }
 
-    public void setShowLint( String warnings )
+
+
+    public void setShowLint( boolean showLint )
     {
-        this.warnings = warnings;
+        this.showLint = showLint;
+    }
+
+    public boolean isShowLint()
+    {
+        return this.showLint;
     }
 
     public void setShowDeprecation( boolean showDeprecation )

plexus-compiler-its/src/main/it/simple-javac/pom.xml

@@ -50,6 +50,7 @@
         <configuration>
           <target>8</target>
           <source>8</source>
+<!--          <showLint>true</showLint>-->
           <compilerArgs>
             <compilerArg>-Xlint:-path</compilerArg>
           </compilerArgs>

plexus-compilers/plexus-compiler-javac/src/main/java/org/codehaus/plexus/compiler/javac/JavacCompiler.java

@@ -343,12 +343,20 @@ public static String[] buildCompilerArguments( CompilerConfiguration config, Str
         if ( !config.isShowWarnings() )
         {
             args.add( "-nowarn" );
-        } else
+        }
+        else
         {
             String warnings = config.getWarnings();
-            if (StringUtils.isNotEmpty(warnings))
+            if (config.isShowLint())
             {
-                args.add("-Xlint:" +  warnings);
+                if(config.isShowWarnings() && StringUtils.isNotEmpty(warnings))
+                {
+                    args.add( "-Xlint:" + warnings );
+                }
+                else
+                {
+                    args.add( "-Xlint" );
+                }
             }
         }