<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
Hi Max, looks good, just a couple nit-picky things:<br>
<ul>
<li>Problem section</li>
<ul>
<li>"Sometimes user wants to..." --> "Sometimes a user wants
to..."</li>
</ul>
<li>Compatibility Risk Description</li>
<ul>
<li>"...there is only behavior change..." --> "there is only
a behavior change..."</li>
</ul>
</ul>
<p>I've also added my name as a reviewer<br>
</p>
--Jamil<br>
<br>
<div class="moz-cite-prefix">On 8/22/2018 7:40 AM, Weijun Wang
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:AEFAAA0A-2246-45A5-8646-8A2F194A0C0A@oracle.com">
<pre wrap="">I just realized the CSR is not reviewed yet. Please take a look.
</pre>
<blockquote type="cite">
<pre wrap="">On Aug 6, 2018, at 4:53 PM, Weijun Wang <a class="moz-txt-link-rfc2396E" href="mailto:weijun.wang@oracle.com"><weijun.wang@oracle.com></a> wrote:
Ping again.
Also please take a review at the CSR at <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8208689">https://bugs.openjdk.java.net/browse/JDK-8208689</a>.
Thanks
Max
</pre>
<blockquote type="cite">
<pre wrap="">On Aug 2, 2018, at 10:28 AM, Weijun Wang <a class="moz-txt-link-rfc2396E" href="mailto:weijun.wang@oracle.com"><weijun.wang@oracle.com></a> wrote:
Please take a review at
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~weijun/8201290/webrev.00/">http://cr.openjdk.java.net/~weijun/8201290/webrev.00/</a>
Please note I didn't use srcProviderName as suggested in the bug report. It was designed to be used by the "keytool -importkeystore" command only.
Thanks
Max
</pre>
</blockquote>
<pre wrap="">
</pre>
</blockquote>
<pre wrap="">
</pre>
</blockquote>
<br>
</body>
</html>