svn commit: r1821853 - /commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/internal/introspection/ClassMap.java

[henrib]

Author: henrib
Date: Mon Jan 22 12:24:16 2018
New Revision: 1821853

URL: http://svn.apache.org/viewvc?rev=1821853&view=rev
Log:
JEXL-246:
Quiesced down logging, was missing check on actual signature diff

Modified:
    
commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/internal/introspection/ClassMap.java

Modified: 
commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/internal/introspection/ClassMap.java
URL: 
http://svn.apache.org/viewvc/commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/internal/introspection/ClassMap.java?rev=1821853&r1=1821852&r2=1821853&view=diff
==============================================================================
--- 
commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/internal/introspection/ClassMap.java
 (original)
+++ 
commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/internal/introspection/ClassMap.java
 Mon Jan 22 12:24:16 2018
@@ -285,9 +285,9 @@ final class ClassMap {
                     // add method to byKey cache; do not override
                     MethodKey key = new MethodKey(mi);
                     Method pmi = cache.byKey.putIfAbsent(key, mi);
-                    if (pmi != null && log.isDebugEnabled()) {
+                    if (pmi != null && log.isDebugEnabled() && !key.equals(new 
MethodKey(pmi))) {
                         // foo(int) and foo(Integer) have the same signature 
for JEXL
-                        log.debug("Method "+ pmi + " is already registered, 
key: " + key);
+                        log.debug("Method "+ pmi + " is already registered, 
key: " + key.debugString());
                     }
                 }
             }