You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by pa...@apache.org on 2022/04/19 07:56:53 UTC

[groovy] 08/11: minor refactor: remove some deprecation warnings

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

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

commit 995430769d0f0ff2789955c408705e837ae72676
Author: Paul King <pa...@asert.com.au>
AuthorDate: Mon Apr 18 21:31:32 2022 +1000

    minor refactor: remove some deprecation warnings
---
 src/main/java/groovy/lang/GroovyClassLoader.java                  | 1 +
 src/main/java/org/codehaus/groovy/reflection/ReflectionUtils.java | 1 +
 2 files changed, 2 insertions(+)

diff --git a/src/main/java/groovy/lang/GroovyClassLoader.java b/src/main/java/groovy/lang/GroovyClassLoader.java
index cb154007f2..f472fb7486 100644
--- a/src/main/java/groovy/lang/GroovyClassLoader.java
+++ b/src/main/java/groovy/lang/GroovyClassLoader.java
@@ -387,6 +387,7 @@ public class GroovyClassLoader extends URLClassLoader {
         }
     }
 
+    @SuppressWarnings("deprecation") // TODO replace getPackage with getDefinedPackage once min JDK version >= 9
     private void definePackageInternal(String className) {
         int i = className.lastIndexOf('.');
         if (i != -1) {
diff --git a/src/main/java/org/codehaus/groovy/reflection/ReflectionUtils.java b/src/main/java/org/codehaus/groovy/reflection/ReflectionUtils.java
index f5e662af6e..2510911b3e 100644
--- a/src/main/java/org/codehaus/groovy/reflection/ReflectionUtils.java
+++ b/src/main/java/org/codehaus/groovy/reflection/ReflectionUtils.java
@@ -210,6 +210,7 @@ public class ReflectionUtils {
     }
 
     // to be run in PrivilegedAction!
+    @SuppressWarnings("deprecation") // replace isAccessible with canAccess once min JDK version >= 9
     public static Optional<AccessibleObject> makeAccessible(final AccessibleObject ao) {
         try {
             if (ao.isAccessible() || trySetAccessible(ao)) {