You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by su...@apache.org on 2022/05/12 17:20:28 UTC

[groovy] 01/01: Avoid unnecessary guards for receiver and parameter

This is an automated email from the ASF dual-hosted git repository.

sunlan pushed a commit to branch danielsun/lab-indy-20220512
in repository https://gitbox.apache.org/repos/asf/groovy.git

commit aad0129a4a298da61d9fc96bb5cae93d1f1d3b35
Author: Daniel Sun <su...@apache.org>
AuthorDate: Fri May 13 01:13:23 2022 +0800

    Avoid unnecessary guards for receiver and parameter
---
 src/main/java/org/codehaus/groovy/vmplugin/v8/Selector.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/main/java/org/codehaus/groovy/vmplugin/v8/Selector.java b/src/main/java/org/codehaus/groovy/vmplugin/v8/Selector.java
index 3301c18225..dadeb8d1bb 100644
--- a/src/main/java/org/codehaus/groovy/vmplugin/v8/Selector.java
+++ b/src/main/java/org/codehaus/groovy/vmplugin/v8/Selector.java
@@ -934,7 +934,8 @@ public abstract class Selector {
                 } else {
                     Class<?> argClass = arg.getClass();
                     if (paramType.isPrimitive()) continue;
-                    //if (Modifier.isFinal(argClass.getModifiers()) && TypeHelper.argumentClassIsParameterClass(argClass,pt[i])) continue;
+                    if (Modifier.isFinal(paramType.getModifiers()) && TypeHelper.argumentClassIsParameterClass(argClass, paramType))
+                        continue;
                     test = SAME_CLASS.
                             bindTo(argClass).
                             asType(MethodType.methodType(boolean.class, paramType));