]> git.saurik.com Git - wxWidgets.git/commitdiff
Fix type mismatch in wxLogTrace() call in wxSocket.
authorVadim Zeitlin <vadim@wxwidgets.org>
Mon, 16 Apr 2012 14:19:13 +0000 (14:19 +0000)
committerVadim Zeitlin <vadim@wxwidgets.org>
Mon, 16 Apr 2012 14:19:13 +0000 (14:19 +0000)
Cast socket handle to a 32 bit value as we use it with "%d" (or "%u" now as it
makes more sense) format specifier to avoid problems with type mismatches
under Win64.

Closes #14204.

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@71207 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

src/common/socket.cpp

index 33690a33222cc8ed6262c068c804c69c812fd935..0d619a77c2f239f7b83cf12088cc0fc98d10f8f6 100644 (file)
@@ -1814,7 +1814,11 @@ wxSocketServer::wxSocketServer(const wxSockAddress& addr,
         return;
     }
 
         return;
     }
 
-    wxLogTrace( wxTRACE_Socket, wxT("wxSocketServer on fd %d"), m_impl->m_fd );
+    // Notice that we need a cast as SOCKET is 64 bit under Win64 and that the
+    // cast is safe because a SOCKET is a handle and so limited to 32 (or,
+    // actually, even 24) bit values anyhow.
+    wxLogTrace( wxTRACE_Socket, wxT("wxSocketServer on fd %u"),
+                static_cast<unsigned>(m_impl->m_fd) );
 }
 
 // --------------------------------------------------------------------------
 }
 
 // --------------------------------------------------------------------------