+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: http://cr.openjdk.java.net/~amlu/8183531/webrev.00/
Thanks,
Amy
--- old/test/javax/net/ssl/ciphersuites/ECCurvesconstraints.java
2017-07-04 15:47:20.000000000 +0800
+++ new/test/javax/net/ssl/ciphersuites/ECCurvesconstraints.java
2017-07-04 15:47:20.000000000 +0800
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2016, Oracle and/or its affiliates. All rights
reserved.
+ * Copyright (c) 2016, 2017, Oracle and/or its affiliates. All rights
reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -32,7 +32,7 @@
* @test
* @bug 8148516
* @summary Improve the default strength of EC in JDK
- * @modules jdk.crypyo.ec
+ * @modules jdk.crypto.ec
* @run main/othervm ECCurvesconstraints PKIX
* @run main/othervm ECCurvesconstraints SunX509
*/
