Author: michaelo
Date: Sun Nov 30 12:29:27 2014
New Revision: 1642539

URL: http://svn.apache.org/r1642539
Log:
Several language style improvements in log messages

Modified:
    
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/AbstractPmdViolationCheckMojo.java
    
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/CpdReport.java
    
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/PmdReport.java
    
maven/plugins/trunk/maven-pmd-plugin/src/test/java/org/apache/maven/plugin/pmd/PmdViolationCheckMojoTest.java

Modified: 
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/AbstractPmdViolationCheckMojo.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/AbstractPmdViolationCheckMojo.java?rev=1642539&r1=1642538&r2=1642539&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/AbstractPmdViolationCheckMojo.java
 (original)
+++ 
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/AbstractPmdViolationCheckMojo.java
 Sun Nov 30 12:29:27 2014
@@ -262,7 +262,7 @@ public abstract class AbstractPmdViolati
                 message.append( warningCount ).append( " warning" ).append( 
warningCount > 1 ? "s" : "" );
             }
 
-            message.append( ". For more details see:" ).append( 
outputFile.getAbsolutePath() );
+            message.append( ". For more details see: " ).append( 
outputFile.getAbsolutePath() );
         }
         return message.toString();
     }

Modified: 
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/CpdReport.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/CpdReport.java?rev=1642539&r1=1642538&r2=1642539&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/CpdReport.java
 (original)
+++ 
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/CpdReport.java
 Sun Nov 30 12:29:27 2014
@@ -172,7 +172,7 @@ public class CpdReport
                     result = cpd.getMatches().hasNext();
                     if ( result )
                     {
-                        getLog().debug( "Skipping Report as skipEmptyReport is 
true and there are no CPD issues." );
+                        getLog().debug( "Skipping report since skipEmptyReport 
is true and there are no CPD issues." );
                     }
                 }
             }

Modified: 
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/PmdReport.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/PmdReport.java?rev=1642539&r1=1642538&r2=1642539&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/PmdReport.java
 (original)
+++ 
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/PmdReport.java
 Sun Nov 30 12:29:27 2014
@@ -245,7 +245,8 @@ public class PmdReport
                     result = reportListener.hasViolations();
                     if ( result )
                     {
-                        getLog().debug( "Skipping Report as skipEmptyReport is 
true and there are no PMD violations." );
+                        getLog().debug( "Skipping report since skipEmptyReport 
is true and"
+                                        + "there are no PMD violations." );
                     }
                 }
             }

Modified: 
maven/plugins/trunk/maven-pmd-plugin/src/test/java/org/apache/maven/plugin/pmd/PmdViolationCheckMojoTest.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-pmd-plugin/src/test/java/org/apache/maven/plugin/pmd/PmdViolationCheckMojoTest.java?rev=1642539&r1=1642538&r2=1642539&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-pmd-plugin/src/test/java/org/apache/maven/plugin/pmd/PmdViolationCheckMojoTest.java
 (original)
+++ 
maven/plugins/trunk/maven-pmd-plugin/src/test/java/org/apache/maven/plugin/pmd/PmdViolationCheckMojoTest.java
 Sun Nov 30 12:29:27 2014
@@ -100,7 +100,7 @@ public class PmdViolationCheckMojoTest
         {
             String message = e.getMessage();
             if ( message.contains( "You have 5 PMD violations and 3 warnings." 
) ) {
-                System.out.println( "Caught Expected Message: " + 
e.getMessage() );// expected
+                System.out.println( "Caught expected message: " + 
e.getMessage() );// expected
             } 
             else
             {


Reply via email to