RFR: JDK-8029007, Check src/share/native/sun/misc code for JNI pending exceptions

[roger riggs]

Hi Dan,

One other comment, instead of changing the return type of the 
setStaticIntField
just return and the caller should check for exceptions and return.
See jni.h:  CHECK_EXCEPTION(env)

Roger

On 1/10/2014 11:40 AM, roger riggs wrote:
> Hi Dan,
>
> Just pushed are macros in jni_util.h to do the same function as your 
> new macros.
> Please update to use the common macros instead of introducing new ones.
>
> Style wise, I would avoid mixing binary operators (!) with pointers.
> it is clearer to compare with NULL.  (The CHECK_NULL macro will do the 
> check and return).
>
> (Not a Reviewer)
>
> Thanks, Roger
>
>
>
> On 1/10/2014 1:31 AM, Dan Xu wrote:
>> Hi All,
>>
>> Please review the fix for JNI pending exception issues reported in 
>> jdk-8029007. Thanks!
>>
>> Webrev: http://cr.openjdk.java.net/~dxu/8029007/webrev.00/
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8029007
>>
>> -Dan
>