commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From simonetrip...@apache.org
Subject svn commit: r1187567 - /commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/binder/DefaultRulesBinder.java
Date Fri, 21 Oct 2011 21:55:23 GMT
Author: simonetripodi
Date: Fri Oct 21 21:55:23 2011
New Revision: 1187567

URL: http://svn.apache.org/viewvc?rev=1187567&view=rev
Log:
fixed PMD violation: Avoid empty catch blocks

Modified:
    commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/binder/DefaultRulesBinder.java

Modified: commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/binder/DefaultRulesBinder.java
URL: http://svn.apache.org/viewvc/commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/binder/DefaultRulesBinder.java?rev=1187567&r1=1187566&r2=1187567&view=diff
==============================================================================
--- commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/binder/DefaultRulesBinder.java
(original)
+++ commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/binder/DefaultRulesBinder.java
Fri Oct 21 21:55:23 2011
@@ -41,17 +41,17 @@ final class DefaultRulesBinder
     private final List<ErrorMessage> errors = new ArrayList<ErrorMessage>();
 
     /**
-     * 
+     *
      */
     private final FromBinderRuleSet fromBinderRuleSet = new FromBinderRuleSet();
 
     /**
-     * 
+     *
      */
     private ClassLoader classLoader;
 
     /**
-     * 
+     *
      *
      * @param classLoader
      */
@@ -81,7 +81,7 @@ final class DefaultRulesBinder
         int stackIndex = stackTrace.length - 1;
         while ( element == null && stackIndex > 0 ) // O(n) there's no better
way
         {
-            Class<?> moduleClass = null;
+            Class<?> moduleClass;
             try
             {
                 // check if the set ClassLoader resolves the Class in the StackTrace
@@ -99,6 +99,7 @@ final class DefaultRulesBinder
                 {
                     // Class in the StackTrace can't be found, don't write the file name:line
number detail in the
                     // message
+                    moduleClass = null;
                 }
             }
 
@@ -177,7 +178,7 @@ final class DefaultRulesBinder
     }
 
     /**
-     * 
+     *
      *
      * @return
      */
@@ -187,7 +188,7 @@ final class DefaultRulesBinder
     }
 
     /**
-     * 
+     *
      *
      * @return
      */
@@ -197,7 +198,7 @@ final class DefaultRulesBinder
     }
 
     /**
-     * 
+     *
      *
      * @return
      */
@@ -207,7 +208,7 @@ final class DefaultRulesBinder
     }
 
     /**
-     * 
+     *
      *
      * @return
      */



Mime
View raw message