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/18 12:25:53 UTC

[groovy] 01/02: minor refactor: remove some deprecation warnings

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

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

commit c38a78c371cbed1280d92e9481c306b8822e4fae
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 2c03b16e3f..cb5e5bde92 100644
--- a/src/main/java/groovy/lang/GroovyClassLoader.java
+++ b/src/main/java/groovy/lang/GroovyClassLoader.java
@@ -392,6 +392,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 1268899c02..dc390437d4 100644
--- a/src/main/java/org/codehaus/groovy/reflection/ReflectionUtils.java
+++ b/src/main/java/org/codehaus/groovy/reflection/ReflectionUtils.java
@@ -211,6 +211,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)) {