Re: JDK 10 RFR of JDK-8183531: Wrong @modules in test/javax/net/ssl/ciphersuites/ECCurvesconstraints.java

2017-07-04 Thread Bradford Wetmore
+1. Brad On 7/4/2017 7:13 AM, Xuelei Fan wrote: Looks fine to me. Thanks, Xuelei On 7/4/2017 12:54 AM, Amy Lu wrote: Please review the patch to fix the typo in @modules for test javax/net/ssl/ciphersuites/ECCurvesconstraints.java bug: https://bugs.openjdk.java.net/browse/JDK-8183531 webrev

Re: JDK 10 RFR of JDK-8183531: Wrong @modules in test/javax/net/ssl/ciphersuites/ECCurvesconstraints.java

2017-07-04 Thread Xuelei Fan
Looks fine to me. Thanks, Xuelei On 7/4/2017 12:54 AM, Amy Lu wrote: Please review the patch to fix the typo in @modules for test javax/net/ssl/ciphersuites/ECCurvesconstraints.java bug: https://bugs.openjdk.java.net/browse/JDK-8183531 webrev: http://cr.openjdk.java.net/~amlu/8183531/webrev.00

AccessController.doPrivileged and default methods

2017-07-04 Thread Michael Rasmussen
Hi I was playing around with having an easy way to have doPrivileged call void methods via lambdas/method references, and noticed that if you use AccessController.doPrivileged(PrivilegedAction), and the run method is implemented as a default method, then you get the following exception: java.lang.

JDK 10 RFR of JDK-8183531: Wrong @modules in test/javax/net/ssl/ciphersuites/ECCurvesconstraints.java

2017-07-04 Thread Amy Lu
Please review the patch to fix the typo in @modules for test javax/net/ssl/ciphersuites/ECCurvesconstraints.java bug: https://bugs.openjdk.java.net/browse/JDK-8183531 webrev: http://cr.openjdk.java.net/~amlu/8183531/webrev.00/ Thanks, Amy --- old/test/javax/net/ssl/ciphersuites/ECCurvesconstrai