<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p><tt>Actually, looking at this current code in JDK 14 (default
branch), it uses the "compareToIgnoreCase" version. So I guess,
you did this change to match with what's already there. So
please ignore my previous comment.</tt></p>
<p><tt>-Jaikiran</tt><br>
</p>
<div class="moz-cite-prefix">On 19/07/19 12:18 PM, Jaikiran Pai
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:d15f36e4-5ebb-9d27-af5c-e2a0ce85e01f@gmail.com">
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
<p><tt>Hello Tony,</tt></p>
<p><tt>I'm not a reviewer, but just checking - is this following
change intentional:</tt></p>
<pre>- statelessSession = Boolean.parseBoolean(st);
+ if (st.compareToIgnoreCase("false") == 0) {
+ statelessSession = false;
+ }
The Boolean.parseBoolean, that's in JDK 13, should work here too.
-Jaikiran
</pre>
<div class="moz-cite-prefix">On 18/07/19 11:19 PM, Anthony
Scarpino wrote:<br>
</div>
<blockquote type="cite"
cite="mid:8f59c04a-50df-74ce-a607-ef50013b0c51@oracle.com">This
is a changeset for preventing Stateless Resumption from being
turned off when JDK-8227551 is merged from 13 into the mainline
repo. This is the way the gatekeepers asked me to do it as
oppose to withholding 8227551 from the mainline merge. <br>
<br>
<a class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/~ascarpino/8228396/webrev/"
moz-do-not-send="true">http://cr.openjdk.java.net/~ascarpino/8228396/webrev/</a>
<br>
<br>
Thanks <br>
<br>
Tony <br>
</blockquote>
</blockquote>
</body>
</html>