[groovy] branch GROOVY_4_0_X updated: Trivial tweak for "GROOVY-10709: Performance regression in Gradle with Groovy 4"

[su...@apache.org]

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

sunlan pushed a commit to branch GROOVY_4_0_X
in repository https://gitbox.apache.org/repos/asf/groovy.git


The following commit(s) were added to refs/heads/GROOVY_4_0_X by this push:
     new bdfedce607 Trivial tweak for "GROOVY-10709: Performance regression in Gradle with Groovy 4"
bdfedce607 is described below

commit bdfedce6078c5091c2220ee53f6db94a6da93366
Author: Daniel Sun <su...@apache.org>
AuthorDate: Sat Aug 6 17:31:19 2022 +0800

    Trivial tweak for "GROOVY-10709: Performance regression in Gradle with Groovy 4"
    
    (cherry picked from commit 27995f4500b7e32ae74e676a90707ab8490fd61f)
---
 src/main/java/org/apache/groovy/lang/GroovyObjectHelper.java | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/apache/groovy/lang/GroovyObjectHelper.java b/src/main/java/org/apache/groovy/lang/GroovyObjectHelper.java
index 23d53e3373..84755ea378 100644
--- a/src/main/java/org/apache/groovy/lang/GroovyObjectHelper.java
+++ b/src/main/java/org/apache/groovy/lang/GroovyObjectHelper.java
@@ -48,6 +48,9 @@ public class GroovyObjectHelper {
         Lookup lookup = lookupAtomicRef.get();
         if (null != lookup) return Optional.of(lookup);
 
+        if (groovyObject.getMetaClass().respondsTo(groovyObject, GET_LOOKUP_METHOD_NAME).isEmpty())
+            return Optional.empty();
+
         if (groovyObjectClass.isMemberClass() && Modifier.isStatic(groovyObjectClass.getModifiers())) {
             List<Class<?>> classList = new ArrayList<>(3);
             for (Class<?> clazz = groovyObjectClass; null != clazz; clazz = clazz.getEnclosingClass()) {
@@ -80,8 +83,6 @@ public class GroovyObjectHelper {
     }
 
     private static Lookup doLookup(GroovyObject groovyObject) {
-        if (groovyObject.getMetaClass().respondsTo(groovyObject, GET_LOOKUP_METHOD_NAME).isEmpty()) return null;
-
         MethodHandles.Lookup lookup;
         try {
             final Class<? extends GroovyObject> groovyObjectClass = groovyObject.getClass();