RFR : JDK-8046545 launcher fix to check function return values

Kumar Srinivasan kumar.x.srinivasan at oracle.com
Fri Jul 18 17:35:39 UTC 2014


Neil,
The fix looks good. However there is an inaccuracy in the comment:

+ *  Normally, JNI calls do not return if an exception is thrown.
+ *  However, this behavior can change in the future,
+ *  so check for thrown exceptions.

This is not true, JNI calls *will* return if an exception is thrown, 
however best
JNI practices dictate that a pending Exception(s) must be cleared or 
caught, before
attempting another JNI call. Under such circumstances the return value 
will usually
be an error or a null value. I suggest making this change to reflect this.

Thanks
Kumar



On 7/18/2014 9:53 AM, Neil Toda wrote:
>
> Please review this launcher change.
>
> Issue: https://bugs.openjdk.java.net/browse/JDK-8046545
> webrev: http://cr.openjdk.java.net/~ntoda/8046545/webrev-01/
>
> Summary:
>
> Introduce a set of macros for launcher to be used to check for certain 
> conditions after
> return from select functions.
>
>




More information about the core-libs-dev mailing list