Thanks for the review. The fix is for 9 not 8. I am waiting for jdk 8
repo to be open.
-Dan
On 12/12/2013 02:49 AM, Michael McMahon wrote:
Yes, looks good Dan. I presume this is going into 9 rather than 8
though...?
Michael.
On 12/12/13 05:14, Dan Xu 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/
<http://cr.openjdk.java.net/%7Edxu/8025437/webrev.00/>
-Dan
