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