<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    Thanks for your comments. Here is the updated patch,
    <meta http-equiv="content-type" content="text/html; charset=UTF-8">
    <a href="http://cr.openjdk.java.net/%7Edxu/7142921/webrev.03/">http://cr.openjdk.java.net/~dxu/7142921/webrev.03/</a>.
    Please review it.<br>
    <br>
    -Dan<br>
    <br>
    <br>
    <div class="moz-cite-prefix">On 11/29/2012 09:54 AM, Alan Bateman
      wrote:<br>
    </div>
    <blockquote cite="mid:50B7A16E.3070900@oracle.com" type="cite">
      <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
      On 29/11/2012 07:45, Dan Xu wrote:
      <blockquote cite="mid:50B7127C.808@oracle.com" type="cite">I have
        updated my fix accordingly and uploaded it to webrev at <a
          moz-do-not-send="true" class="moz-txt-link-freetext"
          href="http://cr.openjdk.java.net/%7Edxu/7142921/webrev.02/">http://cr.openjdk.java.net/~dxu/7142921/webrev.02/</a>.
        Please take a review. Thanks! <br>
        <br>
        -Dan <br>
      </blockquote>
      Thanks for the update.<br>
      <br>
      I see you've removed the protected constructors from Abstract* and
      the public constructors from *FileSystemProvider, I assume you
      didn't mean to do that.<br>
      <br>
      In the getFileSystemDetector methods then I assume that
      System.getProperty("user.name") needs to be done in a privileged
      block, otherwise Files.probeContentType will not work with a
      security manager.<br>
      <br>
      I think the MagicFileTypeDetector looks fine. <br>
      <br>
      <meta http-equiv="content-type" content="text/html; charset=UTF-8">
      MimeTypesFileTypeDetector looks better now that it encapsulates
      just one mime.types files. I think it needs a bit of clean-up
      though, the main issues seems to be that loadMimeTypes is not
      synchronized and there are several potential problems if
      implProbeContentType is invoked by more than one thread at around
      the same time. Also point is that the previous mails I thought
      that testing the extension would be case sensitive only on Linux
      and Solaris.<br>
      <br>
      So overall I think we are a step closer on this.<br>
      <br>
      -Alan.<br>
      <br>
    </blockquote>
    <br>
  </body>
</html>