<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Any comments on the latest rev?<br>
    <br>
    //Nils<br>
    <br>
    Nils Eliasson skrev 2012-03-30 10:40:
    <blockquote cite="mid:4F757183.3080502@oracle.com" type="cite">
      <meta content="text/html; charset=ISO-8859-1"
        http-equiv="Content-Type">
      ok,<br>
      <br>
      New webrev here: <a moz-do-not-send="true"
        href="http://cr.openjdk.java.net/%7Eneliasso/7157695/webrev.03/">http://cr.openjdk.java.net/~neliasso/7157695/webrev.03/</a><br>
      <br>
      Defining socklen_t as int on windows makes a much better impl.<br>
      <br>
      Remains some casts from size_t to int on windows. So we have jni
      call in jvm.h with jint/int -&gt; os.hpp with size_t -&gt;
      os_windows.hpp with int. But we don't expect any negative lenghts
      so it should work.<br>
      <br>
      Thanks for the feedback,<br>
      Nils E<br>
      <br>
      David Holmes skrev 2012-03-29 23:20:
      <blockquote cite="mid:4F74D235.3060500@oracle.com" type="cite">On
        30/03/2012 4:36 AM, Dean Long wrote: <br>
        <blockquote type="cite">Isn't this webrev reversing the recent
          changes for 7091417? <br>
        </blockquote>
        <br>
        Yes - and it shouldn't be. size_t's and socklen_t's are the
        correct types to use. <br>
        <br>
        David <br>
        <br>
        <blockquote type="cite">dl <br>
          <br>
          On 3/29/2012 10:42 AM, Christian Thalinger wrote: <br>
          <blockquote type="cite">I'm pretty sure this gets us into
            trouble with some compiler somewhere <br>
            (I'd guess GCC on Linux). On Mac OS X send is declared as: <br>
            <br>
            ssize_t <br>
            send(int socket, const void *buffer, size_t length, int
            flags); <br>
            <br>
            but length is now passed as int: <br>
            <br>
            -inline int os::send(int fd, char* buf, size_t nBytes, uint
            flags) { <br>
            +inline int os::send(int fd, char* buf, int nBytes, int
            flags) { <br>
            RESTARTABLE_RETURN_INT(::send(fd, buf, nBytes, flags)); <br>
            } <br>
            <br>
            I think your changes are good but we might need some casts
            down in the <br>
            OS files eventually. <br>
            <br>
            -- Chris <br>
            <br>
            On Mar 29, 2012, at 3:00 PM, Nils Eliasson wrote: <br>
            <br>
            <blockquote type="cite"><a moz-do-not-send="true"
                class="moz-txt-link-freetext"
                href="http://cr.openjdk.java.net/%7Eneliasso/7157695/webrev.01/">http://cr.openjdk.java.net/~neliasso/7157695/webrev.01/</a><a
                moz-do-not-send="true" class="moz-txt-link-rfc2396E"
                href="http://cr.openjdk.java.net/%7Eneliasso/7157695/webrev.01/">&lt;http://cr.openjdk.java.net/%7Eneliasso/7157695/webrev.01/&gt;</a>
              <br>
              <br>
              <br>
              7157695 - Add windows implementation of socket interface <br>
              <br>
              Using winsock api which is almost posix compatible. Also
              changing <br>
              some argument types for the socket interface calls in
              os.hpp to make <br>
              the platform independent code more platform independent.
              This made <br>
              the win implementation possible without casts, and allowed
              me to <br>
              remove a lot of redundant casts from the jni exports in
              jvm.cpp. <br>
            </blockquote>
          </blockquote>
        </blockquote>
      </blockquote>
      <br>
      <!-- This signature was generated by the MyDesktop Oracle Business Signature utility version 3.8.8 -->
    </blockquote>
    <br>
    <br>
    <div class="moz-signature">-- <br>
      <a href="http://www.oracle.com" target="_blank"><img
          src="cid:part4.09030105.02090407@oracle.com" alt="Oracle"
          border="0" height="26" width="114"></a><br>
      <font color="#666666" face="Verdana, Arial, Helvetica, sans-serif"
        size="1">Nils Eliasson | Senior Member of Technical Staff<br>
        <font color="#ff0000">Oracle</font> Java Platform Group, JVM
        Engineering<br>
        ORACLE Sweden </font><br>
      <br>
      <!-- This signature was generated by the MyDesktop Oracle Business Signature utility version 3.8.8 -->
    </div>
  </body>
</html>