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 >
