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/08/05 20:50:25 UTC

[groovy] 02/02: Trivial refactoring: extract constant

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

commit 0b8b6e9f6c57ca6353b16363f002e24504744531
Author: Daniel Sun <su...@apache.org>
AuthorDate: Sat Aug 6 04:48:15 2022 +0800

    Trivial refactoring: extract constant
    
    (cherry picked from commit 75339e13ee99562ac8be429c4efbc1fe96db7c9b)
---
 src/main/java/org/apache/groovy/lang/GroovyObjectHelper.java | 6 ++++--
 1 file changed, 4 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 abdee26430..e4dcd4b57c 100644
--- a/src/main/java/org/apache/groovy/lang/GroovyObjectHelper.java
+++ b/src/main/java/org/apache/groovy/lang/GroovyObjectHelper.java
@@ -35,6 +35,8 @@ import java.util.concurrent.atomic.AtomicReference;
  * @since 4.0.0
  */
 public class GroovyObjectHelper {
+    private static final String GET_LOOKUP_METHOD_NAME = "$getLookup";
+
     /**
      * Get the {@link Lookup} instance of the {@link GroovyObject} instance
      *
@@ -80,7 +82,7 @@ public class GroovyObjectHelper {
     }
 
     private static Lookup doLookup(GroovyObject groovyObject) {
-        if (groovyObject.getMetaClass().respondsTo(groovyObject, "$getLookup").isEmpty()) return null;
+        if (groovyObject.getMetaClass().respondsTo(groovyObject, GET_LOOKUP_METHOD_NAME).isEmpty()) return null;
 
         MethodHandles.Lookup lookup;
         try {
@@ -116,7 +118,7 @@ public class GroovyObjectHelper {
     }
 
     private static Method findGetLookupMethod(Class<?> groovyObjectClass) throws NoSuchMethodException {
-        return groovyObjectClass.getDeclaredMethod("$getLookup");
+        return groovyObjectClass.getDeclaredMethod(GET_LOOKUP_METHOD_NAME);
     }
 
     private GroovyObjectHelper() {}