On Thu, 5 Dec 2024 15:16:46 GMT, Chen Liang <li...@openjdk.org> wrote:
>> I had hoped to keep the change focused on the removal of the left over >> ReflectUtil.checkPackageAccess, the else throw AssertionError was a surprise >> to see. So yes, I agree it can be removed but I'd prefer not touch it now, >> future work in this area can remove it. > > Indeed, we should clean this up with things like MethodUtil. MethodUtil is for APIs that do reflective stuff on behave of callers so not tied to the SM execution mode. That is for another time. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22572#discussion_r1871580656
