You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2010/01/27 18:44:34 UTC

svn commit: r903757 - in /commons/proper/digester/trunk/src: java/org/apache/commons/digester/ test/org/apache/commons/digester/

Author: sebb
Date: Wed Jan 27 17:44:33 2010
New Revision: 903757

URL: http://svn.apache.org/viewvc?rev=903757&view=rev
Log:
Unnecessary semi-colons

Modified:
    commons/proper/digester/trunk/src/java/org/apache/commons/digester/CallMethodRule.java
    commons/proper/digester/trunk/src/java/org/apache/commons/digester/Digester.java
    commons/proper/digester/trunk/src/java/org/apache/commons/digester/Rule.java
    commons/proper/digester/trunk/src/test/org/apache/commons/digester/BeanPropertySetterRuleTestCase.java
    commons/proper/digester/trunk/src/test/org/apache/commons/digester/SetPropertiesRuleTestCase.java
    commons/proper/digester/trunk/src/test/org/apache/commons/digester/SetPropertyRuleTestCase.java

Modified: commons/proper/digester/trunk/src/java/org/apache/commons/digester/CallMethodRule.java
URL: http://svn.apache.org/viewvc/commons/proper/digester/trunk/src/java/org/apache/commons/digester/CallMethodRule.java?rev=903757&r1=903756&r2=903757&view=diff
==============================================================================
--- commons/proper/digester/trunk/src/java/org/apache/commons/digester/CallMethodRule.java (original)
+++ commons/proper/digester/trunk/src/java/org/apache/commons/digester/CallMethodRule.java Wed Jan 27 17:44:33 2010
@@ -550,7 +550,6 @@
             // When paramCount is zero and paramTypes.length is zero it
             // means that we truly are calling a method with no parameters.
             // Nothing special needs to be done here.
-            ;
         }
 
         // Construct the parameter values array we will need

Modified: commons/proper/digester/trunk/src/java/org/apache/commons/digester/Digester.java
URL: http://svn.apache.org/viewvc/commons/proper/digester/trunk/src/java/org/apache/commons/digester/Digester.java?rev=903757&r1=903756&r2=903757&view=diff
==============================================================================
--- commons/proper/digester/trunk/src/java/org/apache/commons/digester/Digester.java (original)
+++ commons/proper/digester/trunk/src/java/org/apache/commons/digester/Digester.java Wed Jan 27 17:44:33 2010
@@ -501,7 +501,7 @@
     @Deprecated
     public void setDebug(int debug) {
 
-        ; // No action is taken
+        // No action is taken
 
     }
 
@@ -1413,7 +1413,7 @@
                     new String(buffer, start, len) + ")");
         }
 
-        ;   // No processing required
+        // No processing required
 
     }
 
@@ -1440,7 +1440,7 @@
             saxLog.debug("processingInstruction('" + target + "','" + data + "')");
         }
 
-        ;   // No processing is required
+        // No processing is required
 
     }
 
@@ -1487,7 +1487,7 @@
             saxLog.debug("skippedEntity(" + name + ")");
         }
 
-        ; // No processing required
+        // No processing required
 
     }
 
@@ -3160,7 +3160,7 @@
             try {
                 source.getByteStream().close();
             } catch (IOException e) {
-                ; // Fall through so we get them all
+                // Fall through so we get them all
             }
         }
         inputSources.clear();
@@ -3219,7 +3219,7 @@
     protected void initialize() {
 
         // Perform lazy initialization as needed
-        ; // Nothing required by default
+        // Nothing required by default
 
     }    
 

Modified: commons/proper/digester/trunk/src/java/org/apache/commons/digester/Rule.java
URL: http://svn.apache.org/viewvc/commons/proper/digester/trunk/src/java/org/apache/commons/digester/Rule.java?rev=903757&r1=903756&r2=903757&view=diff
==============================================================================
--- commons/proper/digester/trunk/src/java/org/apache/commons/digester/Rule.java (original)
+++ commons/proper/digester/trunk/src/java/org/apache/commons/digester/Rule.java Wed Jan 27 17:44:33 2010
@@ -151,7 +151,7 @@
     @Deprecated
     public void begin(Attributes attributes) throws Exception {
 
-        ;  // The default implementation does nothing
+        // The default implementation does nothing
 
     }
 
@@ -192,7 +192,7 @@
     @Deprecated
     public void body(String text) throws Exception {
 
-        ;  // The default implementation does nothing
+        // The default implementation does nothing
 
     }
 
@@ -232,7 +232,7 @@
     @Deprecated
     public void end() throws Exception {
 
-        ;  // The default implementation does nothing
+        // The default implementation does nothing
 
     }
 
@@ -265,7 +265,7 @@
      */
     public void finish() throws Exception {
 
-        ;  // The default implementation does nothing
+        // The default implementation does nothing
 
     }
 

Modified: commons/proper/digester/trunk/src/test/org/apache/commons/digester/BeanPropertySetterRuleTestCase.java
URL: http://svn.apache.org/viewvc/commons/proper/digester/trunk/src/test/org/apache/commons/digester/BeanPropertySetterRuleTestCase.java?rev=903757&r1=903756&r2=903757&view=diff
==============================================================================
--- commons/proper/digester/trunk/src/test/org/apache/commons/digester/BeanPropertySetterRuleTestCase.java (original)
+++ commons/proper/digester/trunk/src/test/org/apache/commons/digester/BeanPropertySetterRuleTestCase.java Wed Jan 27 17:44:33 2010
@@ -327,7 +327,7 @@
                 Throwable t =
                     ((InvocationTargetException) e).getTargetException();
                 if (t instanceof NoSuchMethodException) {
-                    ; // Expected result
+                    // Expected result
                 } else {
                     fail("Should have thrown NoSuchMethodException, threw " + t);
                 }

Modified: commons/proper/digester/trunk/src/test/org/apache/commons/digester/SetPropertiesRuleTestCase.java
URL: http://svn.apache.org/viewvc/commons/proper/digester/trunk/src/test/org/apache/commons/digester/SetPropertiesRuleTestCase.java?rev=903757&r1=903756&r2=903757&view=diff
==============================================================================
--- commons/proper/digester/trunk/src/test/org/apache/commons/digester/SetPropertiesRuleTestCase.java (original)
+++ commons/proper/digester/trunk/src/test/org/apache/commons/digester/SetPropertiesRuleTestCase.java Wed Jan 27 17:44:33 2010
@@ -199,7 +199,7 @@
                 Exception ee = ((SAXException) e).getException();
                 if (ee != null) {
                     if (ee instanceof NoSuchMethodException) {
-                        ; // Expected result
+                        // Expected result
                     } else {
                         fail("Should have thrown SE->NoSuchMethodException, threw " + ee);
                     }

Modified: commons/proper/digester/trunk/src/test/org/apache/commons/digester/SetPropertyRuleTestCase.java
URL: http://svn.apache.org/viewvc/commons/proper/digester/trunk/src/test/org/apache/commons/digester/SetPropertyRuleTestCase.java?rev=903757&r1=903756&r2=903757&view=diff
==============================================================================
--- commons/proper/digester/trunk/src/test/org/apache/commons/digester/SetPropertyRuleTestCase.java (original)
+++ commons/proper/digester/trunk/src/test/org/apache/commons/digester/SetPropertyRuleTestCase.java Wed Jan 27 17:44:33 2010
@@ -167,12 +167,12 @@
             assertNotNull(bean); // just to prevent compiler warning on unused var
         } catch (Exception e) {
             if (e instanceof NoSuchMethodException) {
-                ; // Expected result
+                // Expected result
             } else if (e instanceof InvocationTargetException) {
                 Throwable t =
                     ((InvocationTargetException) e).getTargetException();
                 if (t instanceof NoSuchMethodException) {
-                    ; // Expected result
+                    // Expected result
                 } else {
                     fail("Should have thrown ITE->NoSuchMethodException, threw " + t);
                 }
@@ -180,7 +180,7 @@
                 Exception ee = ((SAXException) e).getException();
                 if (ee != null) {
                     if (ee instanceof NoSuchMethodException) {
-                        ; // Expected result
+                        // Expected result
                     } else {
                         fail("Should have thrown SE->NoSuchMethodException, threw " + ee);
                     }