<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">The fix looks fine. Thanks!<br>
      <br>
      On 25.06.15 18:51, Anton Nashatyrev wrote:<br>
    </div>
    <blockquote cite="mid:558C238F.2080701@oracle.com" type="cite">
      <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
      Sergey, Phil<br>
      <br>
          thanks for comments! <br>
      <br>
          here is the updated version: <a moz-do-not-send="true"
        href="http://cr.openjdk.java.net/%7Eanashaty/8080504/webrev.01/">http://cr.openjdk.java.net/~anashaty/8080504/webrev.01/</a><br>
      <br>
      Thanks!<br>
      Anton.<br>
      <br>
      <div class="moz-cite-prefix">On 25.06.2015 0:56, Phil Race wrote:<br>
      </div>
      <blockquote cite="mid:558B2796.8090301@oracle.com" type="cite">
        <meta content="text/html; charset=utf-8"
          http-equiv="Content-Type">
        <div class="moz-cite-prefix">On 06/24/2015 12:26 PM, Sergey
          Bylokhov wrote:<br>
        </div>
        <blockquote cite="mid:558B046D.3080808@oracle.com" type="cite">
          <meta content="text/html; charset=utf-8"
            http-equiv="Content-Type">
          <div class="moz-cite-prefix">Hi, Anton.<br>
             - Why the timeout is used as an int?<br>
          </div>
        </blockquote>
        <br>
        I agree that seems wrong. The downcast from long to int<br>
        could even result in the int being "-1" .. which would be<br>
        the infinite timeout - although I am not sure who would<br>
        wait around to see the difference :-)<br>
        <br>
        <br>
        -phil.<br>
        <br>
        <blockquote cite="mid:558B046D.3080808@oracle.com" type="cite">
          <div class="moz-cite-prefix">  - Will the fix conform to this
            statement in the documentation of ST.realSync()?<br>
            <meta http-equiv="content-type" content="text/html;
              charset=utf-8">
            @param timeout the maximum time to wait in milliseconds,<b>
              negative means "forever"</b>.<br>
            But I am not sure that all our implementations works in case
            of negative timeout.<br>
            <br>
            On 23.06.15 20:31, Anton Nashatyrev wrote:<br>
          </div>
          <blockquote cite="mid:558997EF.7000709@oracle.com" type="cite">Hello,



            <br>
                could you please review the following fix: <br>
            <br>
            fix: <a moz-do-not-send="true"
              class="moz-txt-link-freetext"
              href="http://cr.openjdk.java.net/%7Eanashaty/8080504/webrev.00/">http://cr.openjdk.java.net/~anashaty/8080504/webrev.00/</a>
            <a moz-do-not-send="true" class="moz-txt-link-rfc2396E"
              href="http://cr.openjdk.java.net/%7Eanashaty/8080504/webrev.00/"><http://cr.openjdk.java.net/%7Eanashaty/8080504/webrev.00/></a>
            <br>
            bug: <a moz-do-not-send="true"
              class="moz-txt-link-freetext"
              href="https://bugs.openjdk.java.net/browse/JDK-8080504">https://bugs.openjdk.java.net/browse/JDK-8080504</a>
            <br>
            <br>
                Problem: under MacOS X the realSync() may hang under
            some circumstances. <br>
            <br>
                Fix:  make the 
            Java_sun_lwawt_macosx_LWCToolkit_nativeSyncQueue method
            respecting timeout argument <br>
            <br>
            Thanks! <br>
            Anton. <br>
          </blockquote>
          <br>
          <br>
          <pre class="moz-signature" cols="72">-- 
Best regards, Sergey. </pre>
        </blockquote>
        <br>
      </blockquote>
      <br>
    </blockquote>
    <br>
    <br>
    <pre class="moz-signature" cols="72">-- 
Best regards, Sergey. </pre>
  </body>
</html>