Author: henrib
Date: Sun Jan 21 10:24:49 2018
New Revision: 1821777

URL: http://svn.apache.org/viewvc?rev=1821777&view=rev
Log:
JEXL:
Log signature collision in debug 

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=1821777&r1=1821776&r2=1821777&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
 Sun Jan 21 10:24:49 2018
@@ -283,7 +283,12 @@ final class ClassMap {
                 Method mi = methods[i];
                 if (permissions.allow(mi)) {
                     // add method to byKey cache; do not override
-                    cache.byKey.putIfAbsent(new MethodKey(mi), mi);
+                    MethodKey key = new MethodKey(mi);
+                    Method pmi = cache.byKey.putIfAbsent(key, mi);
+                    if (pmi != null && log.isDebugEnabled()) {
+                        // foo(int) and foo(Integer) have the same signature 
for JEXL
+                        log.debug("Method "+ pmi + " is already registered, 
key: " + key);
+                    }
                 }
             }
         } catch (SecurityException se) {


Reply via email to