[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