<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, 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 href="http://cr.openjdk.java.net/~pchelko/9/8037485/webrev.03/">http://cr.openjdk.java.net/~pchelko/9/8037485/webrev.03/</a></div><div><br></div><div>With best regards. Petr.</div><div><br><div><div>On 23 июля 2014 г., at 17:27, Alan Bateman <<a href="mailto:Alan.Bateman@oracle.com">Alan.Bateman@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">
  
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  
  <div bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">On 23/07/2014 09:25, Petr Pchelko
      wrote:<br>
    </div>
    <blockquote cite="mid:8F600339-7CBA-41D4-A7CB-2560B650CA41@oracle.com" type="cite">
      <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
      Hello, Alan.
      <div><br>
      </div>
      <div>Thank you for the review.</div>
      <div>I've updated the fix according to your comments. The new
        version is here:</div>
      <div><a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Epchelko/9/8037485/webrev.02/">http://cr.openjdk.java.net/~pchelko/9/8037485/webrev.02/</a></div>
      <div><br>
      </div>
      <div>Only the DataFlavorUtil file is updated. DesktopService now
        uses a lazy holder, the doc is fixed, the getFlavorTable method
        is fixed. </div>
      <div>All the rest is the same as in the previous version.</div>
    </blockquote>
    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();<br>
    <br>
    -Alan<br>
    <br>
    <br>
  </div>

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