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

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


The following commit(s) were added to refs/heads/master by this push:
     new 560dbdd7e2 org.apache.juneau.common.reflect API improvements
560dbdd7e2 is described below

commit 560dbdd7e231f8a81befd29d2b72f3d6f45663ab
Author: James Bognar <[email protected]>
AuthorDate: Tue Nov 18 12:05:46 2025 -0500

    org.apache.juneau.common.reflect API improvements
---
 .../org/apache/juneau/common/reflect/ClassInfo.java     |  2 +-
 .../apache/juneau/common/reflect/ConstructorInfo.java   | 17 -----------------
 .../juneau/common/reflect/ConstructorInfoTest.java      |  2 +-
 3 files changed, 2 insertions(+), 19 deletions(-)

diff --git 
a/juneau-core/juneau-common/src/main/java/org/apache/juneau/common/reflect/ClassInfo.java
 
b/juneau-core/juneau-common/src/main/java/org/apache/juneau/common/reflect/ClassInfo.java
index ef226998d1..0696c322a4 100644
--- 
a/juneau-core/juneau-common/src/main/java/org/apache/juneau/common/reflect/ClassInfo.java
+++ 
b/juneau-core/juneau-common/src/main/java/org/apache/juneau/common/reflect/ClassInfo.java
@@ -960,7 +960,7 @@ public class ClassInfo extends ElementInfo implements 
Annotatable {
                return getDeclaredConstructors().stream()
                        .filter(cc -> cc.hasNumParameters(expectedParams))
                        .filter(cc -> cc.isVisible(v))
-                       .map(cc -> cc.accessible(v))
+                       .map(cc -> cc.accessible())
                        .findFirst();
        }
 
diff --git 
a/juneau-core/juneau-common/src/main/java/org/apache/juneau/common/reflect/ConstructorInfo.java
 
b/juneau-core/juneau-common/src/main/java/org/apache/juneau/common/reflect/ConstructorInfo.java
index 5b4caec8e6..c579c3b500 100644
--- 
a/juneau-core/juneau-common/src/main/java/org/apache/juneau/common/reflect/ConstructorInfo.java
+++ 
b/juneau-core/juneau-common/src/main/java/org/apache/juneau/common/reflect/ConstructorInfo.java
@@ -69,23 +69,6 @@ public class ConstructorInfo extends ExecutableInfo 
implements Comparable<Constr
                return this;
        }
 
-       /**
-        * Makes constructor accessible if it matches the visibility 
requirements, or returns <jk>null</jk> if it doesn't.
-        *
-        * <p>
-        * Security exceptions thrown on the call to {@link 
Constructor#setAccessible(boolean)} are quietly ignored.
-        *
-        * @param v The minimum visibility.
-        * @return
-        *      The same constructor if visibility requirements met, or 
<jk>null</jk> if visibility requirement not
-        *      met or call to {@link Constructor#setAccessible(boolean)} 
throws a security exception.
-        */
-       public ConstructorInfo accessible(Visibility v) {
-               if (v.transform(inner) == null)
-                       return null;
-               return this;
-       }
-
        @Override
        public int compareTo(ConstructorInfo o) {
                int i = getSimpleName().compareTo(o.getSimpleName());
diff --git 
a/juneau-utest/src/test/java/org/apache/juneau/common/reflect/ConstructorInfoTest.java
 
b/juneau-utest/src/test/java/org/apache/juneau/common/reflect/ConstructorInfoTest.java
index 0861578107..1e82fda3b2 100644
--- 
a/juneau-utest/src/test/java/org/apache/juneau/common/reflect/ConstructorInfoTest.java
+++ 
b/juneau-utest/src/test/java/org/apache/juneau/common/reflect/ConstructorInfoTest.java
@@ -118,7 +118,7 @@ class ConstructorInfoTest extends TestBase {
        }
 
        @Test void accessible() throws Exception {
-               b_c3.accessible(Visibility.PROTECTED);
+               b_c3.accessible();
                assertEquals(null, b_c3.newInstanceLenient(123).toString());
        }
 

Reply via email to