[Update]: webrev request: JDK-6996377

Xuelei Fan xuelei.fan at oracle.com
Fri May 9 02:00:22 UTC 2014


test/sun/security/validator/ConstructorTest.java
------------------------------------------------
Missed a "," in the copyright date line.  May only need 2014 as this is
a new test.

-  * Copyright (c) 2010, 2014 Oracle and/or its affiliates. ...
+  * Copyright (c) 2010, 2014, Oracle and/or its affiliates. ...
Or
+  * Copyright (c) 2014, Oracle and/or its affiliates. ...

I would prefer the later.

Otherwise, looks fine to me.

Xuelei

On 5/9/2014 9:23 AM, Jamil Nimeh wrote:
> Hello all,
> 
> Updated webrev to account for Sean and Xuelei's comments is here:
> 
> http://cr.openjdk.java.net/~ascarpino/6996377/webrev.02/
> <http://cr.openjdk.java.net/%7Eascarpino/6996377/webrev.02/>
> 
> Thank you,
> --Jamil
> 
> On 05/07/2014 12:12 PM, Jamil Nimeh wrote:
>> Please review the webrev for JDK-6996377 when you get a chance.
>>
>> http://cr.openjdk.java.net/~ascarpino/6996377/webrev.01/
>>
>> Thank you,
>> --Jamil
> 




More information about the security-dev mailing list