<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<p>Thanks for the comments Brad and Bernd. I've incorporated them
into a new webrev : <br>
</p>
<p><a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~coffeys/webrev.8173783.jdk9.v2/webrev/">http://cr.openjdk.java.net/~coffeys/webrev.8173783.jdk9.v2/webrev/</a></p>
<p>Brad - I've tweaked your suggestion further to indicate if the
default values were used.</p>
<p>
<blockquote type="cite">
<pre style="color: rgb(0, 0, 0); font-style: normal; font-variant-ligatures: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: 2; text-align: start; text-indent: 0px; text-transform: none; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><span class="changed" style="color: blue;">! "Initialized [jdk.tls.namedGroups|default] list contains " +</span>
<span class="changed" style="color: blue;">! "no available elliptic curves. " +</span>
<span class="changed" style="color: blue;">! (property != null ? "(" + property + ")" : "[Default]"));</span></pre>
</blockquote>
</p>
<p>Hope to push once build & test results are back.</p>
<p>regards,<br>
Sean.<br>
</p>
<br>
<div class="moz-cite-prefix">On 07/02/2017 19:15, Bradford Wetmore
wrote:<br>
</div>
<blockquote
cite="mid:fb21957f-98a1-4efc-6d9b-612d8ef0b8c0@oracle.com"
type="cite">Nit, I don't like the wording of 196-7. You have
either jdk.tls.namedGroups or if not defined, you have the default
values.
<br>
<br>
In that case, this is going to print "Property defined: null"
<br>
<br>
Maybe something along the lines of:
<br>
<br>
"Initialized [jdk.tls.namedGroups|default] list contains " +
<br>
"no available elliptic curves." +
<br>
(property.exists() ? " " + property : "")
<br>
<br>
where [jdk.tls.namedGroups|default] is the branch you took to
arrive at the list value.
<br>
<br>
Brad
<br>
<br>
<br>
On 2/7/2017 7:25 AM, Seán Coffey wrote:
<br>
<blockquote type="cite">The recent JDK-8148516 enhancement causes
issue for JDKs without EC
<br>
support. It's primarily an issue for JDK 6u which doesn't have
SunEC but
<br>
this still needs to be fixed in all release families.
<br>
<br>
bug report : <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8173783">https://bugs.openjdk.java.net/browse/JDK-8173783</a>
<br>
webrev :
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~coffeys/webrev.8173783.jdk9/webrev/">http://cr.openjdk.java.net/~coffeys/webrev.8173783.jdk9/webrev/</a>
<br>
<br>
regards,
<br>
Sean.
<br>
<br>
</blockquote>
</blockquote>
<br>
</body>
</html>