Hi Vyom, Thanks for taking care of this issue. This looks good to me as well.
I can sponsor this change for you. best regards, -- daniel On 01/09/15 10:38, Vyom Tewari wrote:
Hi everyone, Can you please review my changes for below bug. Bug: JDK-8064470 : JNI exception pending in jdk/src/java/base/unix/native/libjava/FileDescriptor_md.c Webrev: http://cr.openjdk.java.net/~dfuchs/vyom/8064470/webrev.01/ This change ensure that we immediately return if an exception has been raised (result == NULL) by GetMethodId() or proceed to the next line if there is no JNI pending exception. if result is NULL, there is a pending exception, and we return. if result is not NULL, there is no pending exception, and we proceed to the next line... Thanks, Vyom