Re: RFR: Checking for JNI Exceptions in jni_util.c

[roger riggs]

Hi Alan,

Thanks for the review and suggestions; the webrev has been updated
with the recommendations.

Webrev:
  http://cr.openjdk.java.net/~rriggs/webrev-check_exception-8030993/

Thanks, Roger

On 2/4/2014 10:12 AM, Alan Bateman wrote:

On 04/02/2014 14:33, roger riggs wrote:

Please review these cleanups to check for JNI pending exceptions in jni_util.c.

Webrev:
http://cr.openjdk.java.net/~rriggs/webrev-check_exception-8030993/

This mostly looks okay, I just wonder about the jclass declarations at L686 and 690 as I assume these are declaration-after-use warnings with some compilers.

Also there is JNU_ClassString use at L760 that I assume should be replaced with strClazz.

-Alan.