Done. Here's the updated webrev http://cr.openjdk.java.net/~jpai/webrev/8226976/02/webrev/
-Jaikiran On 02/07/19 11:03 AM, Xuelei Fan wrote: > The copyright date format is similar to “.. 2001, 2019, ..”. > > Xuelei > > On Jul 1, 2019, at 10:22 PM, Jaikiran Pai <jai.forums2...@gmail.com > <mailto:jai.forums2...@gmail.com>> wrote: > >> >> On 02/07/19 7:35 AM, Jaikiran Pai wrote: >>> Thank you Xuelei. Here's the updated webrev >>> http://cr.openjdk.java.net/~jpai/webrev/8226976/01/webrev/. >> - * Copyright (c) 2001, 2017, Oracle and/or its affiliates. All rights >> reserved. >> + * Copyright (c) 2001, 2017, 2019 Oracle and/or its affiliates. All rights >> reserved. >> >> I just checked some other files within the JDK and can't seem to find >> a copyright with more than 2 years listed. So if you meant that I >> should change it to "...2001, 2019..." then please do let me know and >> I'll update it accordingly and send the updated webrev. >> >> -Jaikiran >> >> >>> Updated the >>> copyright year and also added you as the reviewer, in the commit log >>> message. >>> >>> -Jaikiran >>> >>> On 01/07/19 10:17 PM, Xuelei Fan wrote: >>>> Hi Jaikiran, >>>> >>>> Would you mind update the copyright date to 2019? Otherwise, looks >>>> good to me. >>>> >>>> Thanks, >>>> Xuelei >>>> >>>> On 6/29/2019 6:03 AM, Jaikiran Pai wrote: >>>>> While investigating a potential regression in JDK 13+, I happened to use >>>>> the test/jdk/javax/net/ssl/SSLSession/SessionTimeOutTests.java. This >>>>> test incorrectly uses the == operator for one of the String checks. I >>>>> have filed a bug[1] with the details and have a patch with the fix >>>>> here[2]. >>>>> >>>>> Can I please get a review and a sponsor for this change? >>>>> >>>>> [1] https://bugs.openjdk.java.net/browse/JDK-8226976 >>>>> >>>>> [2] http://cr.openjdk.java.net/~jpai/webrev/8226976/0/webrev/ >>>>> >>>>> -Jaikiran >>>>> >>>>>