8031494: [launcher] java launcher should check for JNI Pending exceptions.
Kumar Srinivasan
kumar.x.srinivasan at oracle.com
Tue Jan 14 18:17:35 UTC 2014
Good catch on the CallStaticVoidMethod, fixed these as well, here is the
updated webrev:
http://cr.openjdk.java.net/~ksrini/8031494/webrev.2/
delta webrev to the last:
http://cr.openjdk.java.net/~ksrini/8031494/webrev.2/webrev.delta/index.html
Thanks
Kumar
>
> On 1/13/2014 6:35 PM, Kumar Srinivasan wrote:
>> Mandy,
>>
>>
>>> On 1/13/2014 2:44 PM, Kumar Srinivasan wrote:
>>>> The full webrev:
>>>> http://cr.openjdk.java.net/~ksrini/8031494/webrev.1/
>>>
>>> This looks good. Should it also check if there is any pending
>>> exception after the call to CallStaticVoidMethod that has no return
>>> value? It's fine with me if you want to follow up the calls to
>>> CallStaticVoidMethod in a separate bug.
>>
>> No not necessary, there are already guards for pending exceptions
>> from CallStaticVoidMethod,
>> all of them are in JavaMain, and these macros end with "LEAVE".
>
> I'm referring to multiple calls to CallStaticVoidMethod within the
> same function e.g. PrintUsage (maybe that's the only one) where I
> believe no LEAVE macro call in between.
>
> Mandy
More information about the core-libs-dev
mailing list