<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <pre><span class="changed">Hi Sergey,

javax/print/Doc.java
52  *   {@link javax.print.attribute} should be {@link javax.print.attribute javax.print.attribute}, I guess</span>
</pre>
    <pre>83  * interface Doc should be interface {@code Doc}

javax/print/DocFlavor.java

<span class="changed">347  * {@link java.awt.datatransfer.DataFlavor}. should be </span><span class="changed">{@link java.awt.datatransfer.DataFlavor DataFlavor}

</span><span class="changed">437      * @throws NullPointerException if {@code mimeType} or {@code className} are 
should be "is"</span>

 
<span class="changed">31  * condition involving a doc flavor or flavors (class {@link DocFlavor} same as line 347

javax/print/MimeType.java
Javadoc is added for this method. Why something similar is not added for other public methods?
</span><span class="new">124         /**</span>
<span class="new"> 125          * Constructs a new parameter map entry.</span>
<span class="new"> 126          *

javax/print/MultiDocPrintService.java
</span><span class="changed">31  * capabilities of a {@code Printer} should not use {@code Printer} as it is not a class.

javax/print/PrintService.java
</span><span class="changed">36  * {@code PrintService} describes the capabilities of a {@code Printer} .....same as previous 

javax/print/PrintServiceLookup.java
it's a private method. Do we need this javadoc?
</span><span class="new">454     /**</span>
<span class="new"> 455      * Locates {@code MultiDoc} print {@code Services} capable of printing</span>
<span class="new"> 456      * {@code MultiDocs} containing all the specified doc flavors.</span>
<span class="new"> 457      *

javax/print/ServiceUI.java
</span><span class="changed">147      *         attributes is {@code null}, or the initial PrintService 
should have {@code PrintService}

javax/print/StreamPrintService.java
</span><span class="changed"> 47  * output in a format useful in other contexts. StreamPrintService's
should have {@code StreamPrintService}

This is what I have looked so far.
</span></pre>
    <div class="moz-cite-prefix">Regards<br>
      Prasanta<br>
      On 7/17/2017 5:12 AM, Sergey Bylokhov wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:1ac217d8-6764-b9b0-c942-ccfc7ffc6290@oracle.com">Hello,
      <br>
      Please review the fix for jdk10.
      <br>
      The cleanup was done in the same way as for datatransfer, sound
      and accessibility packages(see links in the CR).
      <br>
      <br>
      I suggest to check the specdiff first, because for some methods
      the specification was reworked. CSR will be filed after technical
      review.
      <br>
      <br>
      Bug: <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>
      Webrev can be found at:
      <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~serb/8184435/webrev.07">http://cr.openjdk.java.net/~serb/8184435/webrev.07</a>
      <br>
      Specdiff:
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~serb/8184435/specdiff.07/overview-summary.html">http://cr.openjdk.java.net/~serb/8184435/specdiff.07/overview-summary.html</a><br>
      <br>
      In this fix the javadoc is updated and the next rules were
      applied:
      <br>
       - <tag> should be replaced by {@tag }
      <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 "null"/"true"/"false"/"this"/"ClassName" should be wrapped
      in {@code } when necessary
      <br>
       - the order of different tags were unified across the package
      <br>
      ... etc
      <br>
      <br>
      There are also some mixing of different "reference usage", for
      example "InputStream" vs "input stream", "String" vs "string",
      etc. I tried to fix some of them.
      <br>
      <br>
    </blockquote>
    <br>
  </body>
</html>