We hollowed out ReflectUtil as one of the early steps when removing the code 
for running in the SecurityManager execution mode. Most of the usages have now 
been removed so the empty (and unused) methods can be removed. FieldUtils and 
ConstructorUtils can be removed too.

ObjectInputStream/ObjectOutputStream has a left over package access check for 
the subclassing case that can be removed.

sun.reflect.generics.reflectiveObjects.TypeVariableImpl.getGenericDeclaration 
has a left over package access check that can be removed. I've changed the 
"should not happen" case to be an assert for now but it's in the wrong place. 
If we have a JDK bug in this area then it should be caught at construction 
time, not by the accessor method. This PR is focused on removing the use of 
ReflectUtil so don't want to do any more here.

The changes for java.management missed a usage ConstructorUtil.getConstructor 
in MBeanInstantiator.findConstructor. This is replaced, to allow 
ConstructorUtils be removed.

Testing: tier1-5

-------------

Commit messages:
 - Initial commit

Changes: https://git.openjdk.org/jdk/pull/22572/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=22572&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8345565
  Stats: 256 lines in 10 files changed: 0 ins; 251 del; 5 mod
  Patch: https://git.openjdk.org/jdk/pull/22572.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/22572/head:pull/22572

PR: https://git.openjdk.org/jdk/pull/22572

Reply via email to