You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by de...@apache.org on 2011/05/01 16:10:27 UTC

svn commit: r1098310 - in /maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes: ChangesValidatorMojo.java schema/XmlValidationHandler.java

Author: dennisl
Date: Sun May  1 14:10:27 2011
New Revision: 1098310

URL: http://svn.apache.org/viewvc?rev=1098310&view=rev
Log:
[MCHANGES-210] Update to Java 5

- Start using generics in ChangesValidatorMojo and XmlValidationHandler

Modified:
    maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesValidatorMojo.java
    maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/schema/XmlValidationHandler.java

Modified: maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesValidatorMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesValidatorMojo.java?rev=1098310&r1=1098309&r2=1098310&view=diff
==============================================================================
--- maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesValidatorMojo.java (original)
+++ maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesValidatorMojo.java Sun May  1 14:10:27 2011
@@ -20,7 +20,6 @@ package org.apache.maven.plugin.changes;
  */
 
 import java.io.File;
-import java.util.Iterator;
 import java.util.List;
 
 import org.apache.maven.plugin.AbstractMojo;
@@ -29,7 +28,7 @@ import org.apache.maven.plugin.MojoFailu
 import org.apache.maven.plugin.changes.schema.ChangesSchemaValidator;
 import org.apache.maven.plugin.changes.schema.SchemaValidatorException;
 import org.apache.maven.plugin.changes.schema.XmlValidationHandler;
-import org.xml.sax.SAXException;
+import org.xml.sax.SAXParseException;
 
 /**
  * 
@@ -114,13 +113,12 @@ public class ChangesValidatorMojo
         }
     }
 
-    private void logSchemaValidation( List /*SAXException*/errors )
+    private void logSchemaValidation( List<SAXParseException> errors )
     {
         getLog().warn( "failed to validate changes.xml file " + xmlPath.getAbsolutePath() );
         getLog().warn( "validation errors: " );
-        for ( Iterator iterator = errors.iterator(); iterator.hasNext(); )
+        for ( SAXParseException error : errors )
         {
-            SAXException error = (SAXException) iterator.next();
             getLog().warn( error.getMessage() );
         }
     }

Modified: maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/schema/XmlValidationHandler.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/schema/XmlValidationHandler.java?rev=1098310&r1=1098309&r2=1098310&view=diff
==============================================================================
--- maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/schema/XmlValidationHandler.java (original)
+++ maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/schema/XmlValidationHandler.java Sun May  1 14:10:27 2011
@@ -38,11 +38,11 @@ public class XmlValidationHandler
    
     private boolean parsingError = false;
 
-    private List /* SAXParseException */errors = new ArrayList();
+    private List<SAXParseException> errors = new ArrayList<SAXParseException>();
 
-    private List /* SAXParseException */fatalErrors = new ArrayList();
+    private List<SAXParseException> fatalErrors = new ArrayList<SAXParseException>();
 
-    private List /* SAXParseException */warnings = new ArrayList();
+    private List<SAXParseException> warnings = new ArrayList<SAXParseException>();
 
     private boolean failOnValidationError;
 
@@ -115,32 +115,32 @@ public class XmlValidationHandler
         this.parsingError = error;
     }
 
-    public List /* SAXParseException */getErrors()
+    public List<SAXParseException> getErrors()
     {
         return errors;
     }
 
-    public void setErrors( List errors )
+    public void setErrors( List<SAXParseException> errors )
     {
         this.errors = errors;
     }
 
-    public List /* SAXParseException */getFatalErrors()
+    public List<SAXParseException> getFatalErrors()
     {
         return fatalErrors;
     }
 
-    public void setFatalErrors( List fatalErrors )
+    public void setFatalErrors( List<SAXParseException> fatalErrors )
     {
         this.fatalErrors = fatalErrors;
     }
 
-    public List /* SAXParseException */ getWarnings()
+    public List<SAXParseException> getWarnings()
     {
         return warnings;
     }
 
-    public void setWarnings( List warnings )
+    public void setWarnings( List<SAXParseException> warnings )
     {
         this.warnings = warnings;
     }