On Wed, 13 Nov 2024 15:57:28 GMT, David M. Lloyd <d...@openjdk.org> wrote:
>> src/java.base/share/classes/jdk/internal/reflect/ReflectionFactory.java line
>> 44:
>>
>>> 42: import java.lang.reflect.Modifier;
>>> 43: import java.lang.reflect.Proxy;
>>> 44: import java.security.PrivilegedAction;
>>
>> Unneeded import.
>
> This is used down below on line 423:
>
>
> private static boolean hasDefaultOrNoSerialization(Class<?> cl) {
> return ! Serializable.class.isAssignableFrom(cl)
> || cl.isInterface()
> || cl.isArray()
> || Proxy.isProxyClass(cl) // <=== right here
> || Externalizable.class.isAssignableFrom(cl)
> || cl.isEnum()
> || cl.isRecord()
> || cl.isHidden()
> || nonSerializableLeafClasses.contains(cl);
> }
Sorry, I meant to flag PrivilegedAction.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19702#discussion_r1840694107
