<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    Hi Alan,<br>
    <br>
    Your suggestion makes the code much leaner. I have updated the fix
    at,
    <meta http-equiv="content-type" content="text/html;
      charset=ISO-8859-1">
    <a href="http://cr.openjdk.java.net/%7Edxu/8002306/webrev.02/">http://cr.openjdk.java.net/~dxu/8002306/webrev.02/</a><br>
    <br>
    Please help review it. Thanks!<br>
    <br>
    -Dan<br>
    <br>
    <br>
    <div class="moz-cite-prefix">On 12/18/2012 01:17 PM, Alan Bateman
      wrote:<br>
    </div>
    <blockquote cite="mid:50D0DD83.3000705@oracle.com" type="cite">
      <meta content="text/html; charset=ISO-8859-1"
        http-equiv="Content-Type">
      On 18/12/2012 19:32, Dan Xu wrote:
      <blockquote cite="mid:50D0C4B1.40807@oracle.com" type="cite">
        <meta content="text/html; charset=ISO-8859-1"
          http-equiv="Content-Type">
        :<br>
        <br>
        I have updated my fix at,
        <meta http-equiv="content-type" content="text/html;
          charset=ISO-8859-1">
        <a moz-do-not-send="true"
          href="http://cr.openjdk.java.net/%7Edxu/8002306/webrev.01/">http://cr.openjdk.java.net/~dxu/8002306/webrev.01/</a>.<br>
        <br>
        It removes checks of null or isConnected, and also addresses a
        potential race situation by using an uninterruptible thread to
        establish the loopback connections, which is the last resort to
        solve the connection failures. In addition, the test is also
        updated to test the pipe directly. Please review the new
        changes. Thanks!<br>
        <br>
        -Dan<br>
      </blockquote>
      I think the approach is good but I'd prefer if we didn't&nbsp;
      duplicate the code to establish the loopback connection. If you
      could put this into own class (that implements Runnable) then you
      could invoke it directly first, and if that is interrupted then
      the background thread can invoke it from its run method.<br>
      <br>
      -Alan<br>
    </blockquote>
    <br>
  </body>
</html>