Looks good to me. Thanks Dan. -Chris.
On 12 Dec 2013, at 05:14, Dan Xu <dan...@oracle.com> wrote: > Hi All, > > Please review the fix for the JNI pending exception issue in > src/solaris/native/sun/net/spi/DefaultProxySelector.c. I have refactored the > code so that JNI calls will only be run when previous JNI calls succeed. > Therefore, it is impossible for an unsafe JNI call to happen. Please help > review it. Thanks! > > Bug: https://bugs.openjdk.java.net/browse/JDK-8025437 > Webrev: http://cr.openjdk.java.net/~dxu/8025437/webrev.01/ > > -Dan