You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by di...@apache.org on 2006/05/01 03:33:44 UTC

svn commit: r398495 - /jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/util/introspection/MethodMap.java

Author: dion
Date: Sun Apr 30 18:33:43 2006
New Revision: 398495

URL: http://svn.apache.org/viewcvs?rev=398495&view=rev
Log:
Checkstyle

Modified:
    jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/util/introspection/MethodMap.java

Modified: jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/util/introspection/MethodMap.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/util/introspection/MethodMap.java?rev=398495&r1=398494&r2=398495&view=diff
==============================================================================
--- jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/util/introspection/MethodMap.java (original)
+++ jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/util/introspection/MethodMap.java Sun Apr 30 18:33:43 2006
@@ -165,27 +165,23 @@
                 Method max = (Method) maximal.next();
 
                 switch (moreSpecific(appArgs, max.getParameterTypes())) {
-                    case MORE_SPECIFIC: {
+                    case MORE_SPECIFIC:
                         /*
                          * This method is more specific than the previously
                          * known maximally specific, so remove the old maximum.
                          */
-
                         maximal.remove();
                         break;
-                    }
 
-                    case LESS_SPECIFIC: {
+                    case LESS_SPECIFIC:
                         /*
                          * This method is less specific than some of the
                          * currently known maximally specific methods, so we
                          * won't add it into the set of maximally specific
                          * methods
                          */
-
                         lessSpecific = true;
                         break;
-                    }
                 }
             }
 
@@ -273,6 +269,9 @@
     /**
      * Returns true if the supplied method is applicable to actual argument
      * types.
+     * @param method the method to check
+     * @param classes possible argument types
+     * @return true if the arguments are applicable to the method.
      */
     private static boolean isApplicable(Method method, Class[] classes) {
         Class[] methodArgs = method.getParameterTypes();



---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org