<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    Looks OK but I am wondering why some of the formatting changes were
    done<br>
    <pre><span class="removed">-     * then a call to {@code getParameter("Color")} returns the</span>
<span class="removed">-     * value {@code "blue"}.</span>
<span class="new">+     * then a call to {@code getParameter("Color")} returns the value</span>
<span class="new">+     * {@code "blue"}.</span></pre>
    <br>
    Seems you were trying to make it so that line breaks were not
    inserted<br>
    gratuitously early ..<br>
    <br>
    This one looks odd :<br>
    <pre><span class="new">+     * Each element of the array should be a set of three {@code Strings}</span></pre>
    <br>
    Since there is no class called "Strings".<br>
    <br>
    Is that how plurals are handled in other parts of the javadoc ?<br>
    <br>
    I thought I might have used {@code String}s instead but I don't know
    how that formats.<br>
    <br>
    -phil.<br>
    <br>
    <div class="moz-cite-prefix">On 10/8/18 12:16 AM, Sergey Bylokhov
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:fe951268-a495-0320-3b57-12e16d5964c3@oracle.com">Hello.
      <br>
      Please review a javadoc "weekend cleanup" in jdk 12.
      <br>
      <br>
      Bug: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8211833">https://bugs.openjdk.java.net/browse/JDK-8211833</a>
      <br>
      Webrev: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~serb/8211833/webrev.00/">http://cr.openjdk.java.net/~serb/8211833/webrev.00/</a>
      <br>
      Specdiff:
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~serb/8211833/specdiff.00/java.desktop/java/applet/package-summary.html">http://cr.openjdk.java.net/~serb/8211833/specdiff.00/java.desktop/java/applet/package-summary.html</a><br>
      <br>
      The next rules were applied:
      <br>
       - 80 column limit
      <br>
       - description of the class/method/field should be followed by dot
      <br>
       - @param, @return should not end with a dot, except a case when
      more than one sentences are used
      <br>
       - empty line after description/before the first tag was added
      <br>
       - unnecessary empty lines were removed
      <br>
       - sets of spaces in the middle of text were deleted
      <br>
       - @param, @throws, @return should be aligned, to be more readable
      <br>
       - unnecessary imports should be removed
      <br>
       - the "true"/"false" should be wrapped in {@code } when necessary
      <br>
       - the order of different tags were unified across the package
      <br>
      <br>
      The styles are the same as was applied to javax.sound.**,
      javax.print, java.datatransfer and javax.accessibility:
      <br>
      <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8184435">https://bugs.openjdk.java.net/browse/JDK-8184435</a>
      <br>
      <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8043979">https://bugs.openjdk.java.net/browse/JDK-8043979</a>
      <br>
      <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8180889">https://bugs.openjdk.java.net/browse/JDK-8180889</a>
      <br>
      <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8181877">https://bugs.openjdk.java.net/browse/JDK-8181877</a>
      <br>
      <br>
      <br>
    </blockquote>
    <br>
  </body>
</html>