You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@commons.apache.org by si...@apache.org on 2011/05/31 10:02:28 UTC

svn commit: r1129547 - /commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/plugins/strategies/FinderFromDfltClass.java

Author: simonetripodi
Date: Tue May 31 08:02:28 2011
New Revision: 1129547

URL: http://svn.apache.org/viewvc?rev=1129547&view=rev
Log:
FinderFromDfltClass members can be final

Modified:
    commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/plugins/strategies/FinderFromDfltClass.java

Modified: commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/plugins/strategies/FinderFromDfltClass.java
URL: http://svn.apache.org/viewvc/commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/plugins/strategies/FinderFromDfltClass.java?rev=1129547&r1=1129546&r2=1129547&view=diff
==============================================================================
--- commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/plugins/strategies/FinderFromDfltClass.java (original)
+++ commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/plugins/strategies/FinderFromDfltClass.java Tue May 31 08:02:28 2011
@@ -39,9 +39,9 @@ public class FinderFromDfltClass
 
     public static String DFLT_METHOD_NAME = "addRules";
 
-    private String rulesClassSuffix;
+    private final String rulesClassSuffix;
 
-    private String methodName;
+    private final String methodName;
 
     /** See {@link #findLoader}. */
     public FinderFromDfltClass()
@@ -99,7 +99,7 @@ public class FinderFromDfltClass
 
         if ( methodName == null )
         {
-            methodName = DFLT_METHOD_NAME;
+            return new LoaderFromClass( rulesClass, DFLT_METHOD_NAME );
         }
 
         return new LoaderFromClass( rulesClass, methodName );