On Thu, 5 Dec 2024 10:20:50 GMT, Alan Bateman <al...@openjdk.org> wrote:
> 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 This pull request has now been integrated. Changeset: 691e6921 Author: Alan Bateman <al...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/691e692149c105b4ca34aaaba779675b6bcc7c65 Stats: 258 lines in 10 files changed: 0 ins; 251 del; 7 mod 8345565: Remove remaining SecurityManager motivated APIs from sun.reflect.util Reviewed-by: mullan, rriggs, liach ------------- PR: https://git.openjdk.org/jdk/pull/22572
