Hi Jc,
On 18/12/2018 3:42 am, JC Beyler wrote:
Hi all,
Could I get a review for this webrev:
Webrev: http://cr.openjdk.java.net/~jcbeyler/8215495/webrev.00/
Bug: https://bugs.openjdk.java.net/browse/JDK-8215495
isCopy only has to be set if the method executes successfully
"If isCopy is not NULL, then *isCopy is set to JNI_TRUE if a copy is
made; or it is set to JNI_FALSE if no copy is made."
You can only make (or not) a copy if the operation actually succeeds. So
before checking isCopy the caller must check for NULL and/or a pending
exception.
I see no bug here.
David
-----
Thanks,
Jc
