<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Hello, <div><br></div><div>Thank you for the review.</div><div>I’ve updated the fix: <a href="http://cr.openjdk.java.net/~pchelko/9/8037485/webrev.04/">http://cr.openjdk.java.net/~pchelko/9/8037485/webrev.04/</a></div><div><br></div><div>I’ve fixed all suggestions except one:</div><div><blockquote type="cite">Another optimization/simplification is possible. You could merge the DesktopDatatransferServiceHolder and default implementation into one class:</blockquote>No, I could not. The default implementation is a member of desktop module while DataFlavorUtil is a part of data transfer module.</div><div><br></div><div><blockquote type="cite"><div bgcolor="#FFFFFF" text="#000000">I skimmed through the webrev and looks good.  It looks like that you have cleanly removed the dependency.  You can run jdk9/bin/jdeps [1] on the java.awt.datatransfer.** and its implementation classes to double check if there is no dependency to the desktop classes.<br></div></blockquote>I’ve run the tool on java.awt.datatransfer and sun.datatransfer packages and there are no dependencies on desktop.</div><div><br></div><div><blockquote type="cite"><div bgcolor="#FFFFFF" text="#000000"><pre>DataFlavorUtil.java
   line 544, 549 - some raw types and you may want to check if there are others. </pre></div></blockquote>Fixed and checked the patch for another raw/unchecked warning. </div><div><br></div><div>With best regards. Petr.</div><div><br></div><div><div><blockquote type="cite"><div>On Jul 23, 2014, at 7:04 PM, Mandy Chung <<a href="mailto:mandy.chung@oracle.com">mandy.chung@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div>
  
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  
  <div bgcolor="#FFFFFF" text="#000000">
    <br>
    <div class="moz-cite-prefix">On 7/23/2014 6:53 AM, Petr Pchelko
      wrote:<br>
    </div>
    <blockquote cite="mid:5032051C-EB18-45B4-B267-4D7BD5D315DC@oracle.com" type="cite">
      <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
      Hello, Alan.
      <div><br>
      </div>
      <div>
        <blockquote type="cite">
          <div bgcolor="#FFFFFF" text="#000000">I'm skimmed over the
            updated webrev, it mostly looks good except for getFlavorMap
            where it doesn't set map, I assume you meant to do this:<br>
            <br>
            if (map == null)<br>
                flavorMap = map = supplier.get();</div>
        </blockquote>
        Thank you! Updated the fix: <a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Epchelko/9/8037485/webrev.03/">http://cr.openjdk.java.net/~pchelko/9/8037485/webrev.03/</a></div>
      <div><br>
      </div>
      <br>
    </blockquote>
    <br>
    I skimmed through the webrev and looks good.  It looks like that you
    have cleanly removed the dependency.  You can run jdk9/bin/jdeps [1]
    on the java.awt.datatransfer.** and its implementation classes to
    double check if there is no dependency to the desktop classes.<br>
    <br>
    Minor comments I spotted:<br>
    <pre>java/awt/datatransfer/SystemFlavorMap.java</pre>
    <pre>line 225 and 238 look like debugging statement to be removed.

DataFlavorUtil.java
   line 544, 549 - some raw types and you may want to check if there are others. 

[1] <a class="moz-txt-link-freetext" href="http://docs.oracle.com/javase/8/docs/technotes/tools/unix/jdeps.html">http://docs.oracle.com/javase/8/docs/technotes/tools/unix/jdeps.html</a>
</pre>
  </div>

</div></blockquote></div><br></div></body></html>