On Mon, 5 Apr 2021 23:15:48 GMT, David Holmes <[email protected]> wrote:
>> Harold Seigel has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> remove unneeded statement
>
> src/hotspot/share/classfile/klassFactory.cpp line 117:
>
>> 115: ClassLoaderData*
>> loader_data,
>> 116: Handle
>> protection_domain,
>> 117:
>> JvmtiCachedClassFileData** cached_class_file) {
>
> I don't think this removal of TRAPS is correct. The ClassFileLoadHook could
> result in an exception becoming pending.
Thanks! This change was reverted.
-------------
PR: https://git.openjdk.java.net/jdk/pull/3345
