On Tue, 4 Feb 2025 14:06:07 GMT, Alan Bateman <al...@openjdk.org> wrote:
>> Thanks for looking at this. I didn't want to change this with this change. > > There may need to be some follow-on cleanup, e.g. I'm wondering if > Lookup.cachedProtectionDomain is needed now. One of the reasons I wanted to move this out of Hotspot as a native call is that it might make further work with ProtectionDomain easier to do all in Java, except there's still a bit of coupling in the JVM with the name of the class and that it's passed through defineClass (resolve_from_stream) and initialized in the mirror. So I guess that's still a lot. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/23396#discussion_r1941345990