On Mon, 5 Apr 2021 23:15:48 GMT, David Holmes <dhol...@openjdk.org> 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

Reply via email to