<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <p>Hi Sergey,<br>
    </p>
    <br>
    <div class="moz-cite-prefix">On 8/12/2017 5:38 AM, Sergey Bylokhov
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:113a3d9b-b948-0b9b-fd5f-1934f7a8a9b9@oracle.com">Hi,
      Prasanta.
      <br>
      Thank you for review!
      <br>
      <br>
      The new version:
      <br>
      <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~serb/8184435/webrev.08/">http://cr.openjdk.java.net/~serb/8184435/webrev.08/</a>
      <br>
      webrev diff v07 vs v08:
      <br>
      <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~serb/8184435/webrev.08/v7_v8.diff">http://cr.openjdk.java.net/~serb/8184435/webrev.08/v7_v8.diff</a>
      <br>
      <br>
      See comments inline.
      <br>
      <br>
      On 07.08.2017 3:29, Prasanta Sadhukhan wrote:
      <br>
      <blockquote type="cite">Hi Sergey, javax/print/Doc.java 52 *
        {@link javax.print.attribute} should be {@link
        javax.print.attribute javax.print.attribute}, I guess
        <br>
      </blockquote>
      <br>
      Both versions generate the same html links so the second part is
      not necessary.
      <br>
      <br>
    </blockquote>
    In that case, will this be required to have 2nd part? <br>
    <pre><span class="changed">43  *   {@link DocFlavor DocFlavor}
</span>
There are manyof them in other files too like DocFlavor.java
<span class="changed">1206      * stream ({@link java.io.Reader java.io.Reader}

Other than that, it looks ok to me (btw, I have not gone through each and every file).

Regards
Prasanta
</span></pre>
    <blockquote type="cite"
      cite="mid:113a3d9b-b948-0b9b-fd5f-1934f7a8a9b9@oracle.com">
      <blockquote type="cite">
        <br>
        83  * interface Doc should be interface {@code Doc}
        <br>
      </blockquote>
      <br>
      Fixed.
      <br>
      <br>
      <blockquote type="cite">
        <br>
        javax/print/DocFlavor.ja >
        <br>
        347 * {@link java.awt.datatransfer.DataFlavor}. should be {@link
        java.awt.datatransfer.DataFlavor DataFlavor}
        <br>
      </blockquote>
      <br>
      The full name is used to highlight that DataFlavor class located
      in other package and it should not be confused with DocFlavor.
      <br>
      <br>
      <br>
      437 * @throws
      <br>
      <blockquote type="cite">NullPointerException if {@code mimeType}
        or {@code className} are should be "is"
        <br>
      </blockquote>
      <br>
      Fixed.
      <br>
      <br>
      <blockquote type="cite">
        <br>
          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? 124 /**
        <br>
        125 * Constructs a new parameter map entry.
        <br>
        126 *
        <br>
      </blockquote>
      <br>
      Most of other methods in this and some other classes have a spec
      from the parent class.
      <br>
      <br>
      <blockquote type="cite">javax/print/MultiDocPrintService.java 31 *
        capabilities of a {@code Printer} should not use {@code Printer}
        as it is not a class. javax/print/PrintService.java 36 * {@code
        PrintService} describes the capabilities of a {@code Printer}
        .....same as previous
        <br>
      </blockquote>
      <br>
      Fixed.
      <br>
      <br>
      <blockquote type="cite">javax/print/PrintServiceLookup.java it's a
        private method. Do we need this javadoc? 454 /**
        <br>
        455 * Locates {@code MultiDoc} print {@code Services} capable of
        printing
        <br>
        456 * {@code MultiDocs} containing all the specified doc
        flavors.
        <br>
      </blockquote>
      <br>
      I added a specs to some private fields/methods , so at some point
      we will be able to enable doclint for everything.
      <br>
      <br>
      <blockquote type="cite">457 * javax/print/ServiceUI.java 147 *
        attributes is {@code null}, or the initial PrintService should
        have {@code PrintService} javax/print/StreamPrintService.java 47
        * output in a format useful in other contexts.
        StreamPrintService's should have {@code StreamPrintService} This
        is what I have looked so far.
        <br>
      </blockquote>
      <br>
      Fixed.
      <br>
      <br>
      <blockquote type="cite">
        <br>
        Regards
        <br>
        Prasanta
        <br>
        On 7/17/2017 5:12 AM, Sergey Bylokhov wrote:
        <br>
        <blockquote type="cite">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>
      </blockquote>
      <br>
      <br>
    </blockquote>
    <br>
  </body>
</html>