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