<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p><br>
</p>
<div class="moz-cite-prefix">On 02/07/19 7:35 AM, Jaikiran Pai
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:c80e228d-4c04-7d04-35c0-56ef42bfa987@gmail.com">
<pre class="moz-quote-pre" wrap="">Thank you Xuelei. Here's the updated webrev
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~jpai/webrev/8226976/01/webrev/">http://cr.openjdk.java.net/~jpai/webrev/8226976/01/webrev/</a>.</pre>
</blockquote>
<pre>- * 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.</pre>
<p>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.</p>
<p>-Jaikiran</p>
<p><br>
</p>
<blockquote type="cite"
cite="mid:c80e228d-4c04-7d04-35c0-56ef42bfa987@gmail.com">
<pre class="moz-quote-pre" wrap=""> 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:
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">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:
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">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] <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8226976">https://bugs.openjdk.java.net/browse/JDK-8226976</a>
[2] <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~jpai/webrev/8226976/0/webrev/">http://cr.openjdk.java.net/~jpai/webrev/8226976/0/webrev/</a>
-Jaikiran
</pre>
</blockquote>
</blockquote>
</blockquote>
</body>
</html>