<html>

  <head>

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

  </head>

  <body text="#000000" bgcolor="#FFFFCC">

    Looks fine.<br>

    <br>

    <div class="moz-cite-prefix">On 3/13/20 11:42 AM, Pavel Rappo wrote:<br>

    </div>

    <blockquote type="cite"

      cite="mid:0AA807B8-EC70-428B-A8B6-9ED6DA0B40EA@oracle.com">

      <pre class="moz-quote-pre" wrap="">Hello,

Please review the change for <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8241014">https://bugs.openjdk.java.net/browse/JDK-8241014</a>:

  <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~prappo/8241014/webrev.00/">http://cr.openjdk.java.net/~prappo/8241014/webrev.00/</a>

This is a documentation cleanup. There are no code changes involved,

and the changes in documentation are mostly trivial.

The following packages are affected:

    java.lang,

    java.nio.file,

    java.nio.file.attribute,

    java.security,

    java.time.chrono,

    java.time.temporal,

    java.util,

    java.util.regex,

    java.util.stream,

    javax.crypto,

    javax.security.cert,

    javax.tools

That said, there are two changes that I'd prefer to be carefully reviewed by

the experts in the corresponding areas.

The first one is for a suspected typo in the javax.crypto.CryptoPolicyParser

class, "AlgrithomParameterSpec". It is not unheard-of for typos to be kept and

supported for the sake of backward compatibility. Sadly, we have a number of

those in OpenJDK. Even though I performed reasonable checks, the proposed fix

should better be verified by the security folk.

The second one is for the doc comment for the java.util.stream.Stream.collect method.

     @apiNote

     The following will accumulate strings into an ArrayList:

         List<String> asList = stringStream.collect(Collectors.toList());

Given that the spec for Collectors.toList() clearly says that

    ...There are no guarantees on the type, mutability, serializability, or

    thread-safety of the List returned;...

I'd assume that @apiNote should be fixed as proposed.

-Pavel

P.S. Apologies for spamming multiple mailing lists. 

</pre>

    </blockquote>

    <br>

  </body>

</html>