<html>

  <head>

    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">

  </head>

  <body>

    <p>Hi Tony,</p>

    <ul>

      <li> "Commonly used and time-constantly implemented curves,

        secp256r1, secp384r1, secp521r1, x25519, and x448, remain

        supported by SunEC" - time-constantly implemented sounds a bit

        awkward to my ear.</li>

      <ul>

        <li>May I suggest an alternative: "The secp256r1, secp384r1,

          secp521r1, x25519, and x448 curve implementations are

          time-constant and remain supported by SunEC."  (I'm sure the

          wording could be better still).</li>

      </ul>

      <li>Why are there asterisks around

        -Djdk.sunec.disableNative=false?  Were you trying to do some

        kind of markdown notation there?</li>

    </ul>

    <p>Otherwise it looks good to me.</p>

    <p>--Jamil<br>

    </p>

    <div class="moz-cite-prefix">On 3/25/2020 4:44 PM, Anthony Scarpino

      wrote:<br>

    </div>

    <blockquote type="cite"

      cite="mid:28156dd9-081f-4ced-1296-6996e9d9ea48@oracle.com">Please

      review the proposed release note for JDK-8237219, the change to

      disable the native curves in SunEC.

      <br>

      <br>

      <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8241386">https://bugs.openjdk.java.net/browse/JDK-8241386</a>

      <br>

      <br>

      Thanks

      <br>

      <br>

      Tony

      <br>

    </blockquote>

  </body>

</html>