This looks good.

In general, it might be more appropriate to review this on net-dev@openjdk.java.net alias.

Sincerely yours,
Ivan


On 02.09.2015 16:22, Vyom Tewari wrote:
Hi everyone,

Can you please review my changes for below bug.

Bug:
JDK-8080486 : JNI exception pending in jdk/src/java.base/windows/native/libnet/DualStackPlainSocketImpl.c

Webrev:
http://cr.openjdk.java.net/~dfuchs/vyom/8080486/webrev.01

This change ensure that we immediately return if an exception has been raised (result == NULL) by GetMethodId/NewGlobalRef or proceed to the next statement if there is no JNI pending exception.

Thanks,
Vyom


Reply via email to