<html>
<head>
<meta http-equiv="Content-Type" content="text/html;
charset=windows-1252">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p>Looks fine to me. I can sponsor this patch if required.</p>
<p>regards,<br>
Sean.<br>
</p>
<div class="moz-cite-prefix">On 03/01/2019 16:51, Roger Calnan
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:D9ED7223-A03F-48CB-8EB1-C2D4EC97CA57@oracle.com">
<meta http-equiv="Content-Type" content="text/html;
charset=windows-1252">
<div class=""><span class="Apple-tab-span" style="white-space:pre"> </span>please
review straightforward fix in the javadoc
for javax.net.ssl.SSLSession.removeValue</div>
<div class=""><br class="">
</div>
<div class=""><span class="Apple-tab-span" style="white-space:pre"> </span>Thanks,</div>
<div class=""><br class="">
</div>
<div class=""><span class="Apple-tab-span" style="white-space:pre"> </span>Roger</div>
<div class=""><br class="">
</div>
<div class=""><b class="" style="font-family: Monaco; font-size:
10px;"><a
href="https://bugs.openjdk.java.net/browse/JDK-8179943"
class="" moz-do-not-send="true">https://bugs.openjdk.java.net/browse/JDK-8179943</a> Typo
in javax.net.ssl.SSLSession.removeValue(String) method
documentation</b></div>
<div class=""><b class=""><br class="" style="font-family: Monaco;
font-size: 10px;">
</b>
<div style="font-family: Monaco; font-size: 10px; margin: 0px;
font-stretch: normal; line-height: normal;" class=""><span
style="font-variant-ligatures: no-common-ligatures;"
class="">diff -r 3d0f6ef91216
src/java.base/share/classes/javax/net/ssl/SSLSession.java</span></div>
<div style="font-family: Monaco; font-size: 10px; margin: 0px;
font-stretch: normal; line-height: normal;" class=""><span
style="font-variant-ligatures: no-common-ligatures;"
class="">---
a/src/java.base/share/classes/javax/net/ssl/SSLSession.java<span class="Apple-tab-span" style="white-space: pre;"> </span>Wed
Jan 02 13:37:55 2019 -0500</span></div>
<div style="font-family: Monaco; font-size: 10px; margin: 0px;
font-stretch: normal; line-height: normal;" class=""><span
style="font-variant-ligatures: no-common-ligatures;"
class="">+++
b/src/java.base/share/classes/javax/net/ssl/SSLSession.java<span class="Apple-tab-span" style="white-space: pre;"> </span>Wed
Jan 02 13:19:38 2019 -0800</span></div>
<div style="font-family: Monaco; font-size: 10px; margin: 0px;
font-stretch: normal; line-height: normal;" class=""><span
style="font-variant-ligatures: no-common-ligatures;"
class="">@@ -1,5 +1,5 @@</span></div>
<div style="font-family: Monaco; font-size: 10px; margin: 0px;
font-stretch: normal; line-height: normal;" class=""><span
style="font-variant-ligatures: no-common-ligatures;"
class=""> /*</span></div>
<div style="font-family: Monaco; font-size: 10px; margin: 0px;
font-stretch: normal; line-height: normal;" class=""><span
style="font-variant-ligatures: no-common-ligatures;"
class="">- * Copyright (c) 1997, 2017, Oracle and/or its
affiliates. All rights reserved.</span></div>
<div style="font-family: Monaco; font-size: 10px; margin: 0px;
font-stretch: normal; line-height: normal;" class=""><span
style="font-variant-ligatures: no-common-ligatures;"
class="">+ * Copyright (c) 1997, 2019, Oracle and/or its
affiliates. All rights reserved.</span></div>
<div style="font-family: Monaco; font-size: 10px; margin: 0px;
font-stretch: normal; line-height: normal;" class=""><span
style="font-variant-ligatures: no-common-ligatures;"
class=""> * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR
THIS FILE HEADER.</span></div>
<div style="font-family: Monaco; font-size: 10px; margin: 0px;
font-stretch: normal; line-height: normal;" class=""><span
style="font-variant-ligatures: no-common-ligatures;"
class=""> *</span></div>
<div style="font-family: Monaco; font-size: 10px; margin: 0px;
font-stretch: normal; line-height: normal;" class=""><span
style="font-variant-ligatures: no-common-ligatures;"
class=""> * This code is free software; you can
redistribute it and/or modify it</span></div>
<div style="font-family: Monaco; font-size: 10px; margin: 0px;
font-stretch: normal; line-height: normal;" class=""><span
style="font-variant-ligatures: no-common-ligatures;"
class="">@@ -187,7 +187,7 @@</span></div>
<div style="font-family: Monaco; font-size: 10px; margin: 0px;
font-stretch: normal; line-height: normal;" class=""><span
style="font-variant-ligatures: no-common-ligatures;"
class=""> * Removes the object bound to the given name
in the session's</span></div>
<div style="font-family: Monaco; font-size: 10px; margin: 0px;
font-stretch: normal; line-height: normal;" class=""><span
style="font-variant-ligatures: no-common-ligatures;"
class=""> * application layer data. Does nothing if
there is no object</span></div>
<div style="font-family: Monaco; font-size: 10px; margin: 0px;
font-stretch: normal; line-height: normal;" class=""><span
style="font-variant-ligatures: no-common-ligatures;"
class=""> * bound to the given name. If the bound
existing object</span></div>
<div style="font-family: Monaco; font-size: 10px; margin: 0px;
font-stretch: normal; line-height: normal;" class=""><span
style="font-variant-ligatures: no-common-ligatures;"
class="">- * implements the {@code
SessionBindingListener} interface,</span></div>
<div style="font-family: Monaco; font-size: 10px; margin: 0px;
font-stretch: normal; line-height: normal;" class=""><span
style="font-variant-ligatures: no-common-ligatures;"
class="">+ * implements the {@code
SSLSessionBindingListener} interface,</span></div>
<div style="font-family: Monaco; font-size: 10px; margin: 0px;
font-stretch: normal; line-height: normal;" class=""><span
style="font-variant-ligatures: no-common-ligatures;"
class=""> * it is notified appropriately.</span></div>
<div style="font-family: Monaco; font-size: 10px; margin: 0px;
font-stretch: normal; line-height: normal;" class=""><span
style="font-variant-ligatures: no-common-ligatures;"
class=""> * <p></span></div>
<div style="font-family: Monaco; font-size: 10px; margin: 0px;
font-stretch: normal; line-height: normal;" class=""><span
style="font-variant-ligatures: no-common-ligatures;"
class=""> * For security reasons, the same named values
may not be</span></div>
</div>
</blockquote>
</body>
</html>