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() {}