Cast -1 to DWORD for comparing to variable
authorThomas Zimmermann <tdz@users.sourceforge.net>
Fri, 19 Aug 2016 09:23:54 +0000 (11:23 +0200)
committerSimon McVittie <simon.mcvittie@collabora.co.uk>
Mon, 10 Oct 2016 14:09:31 +0000 (15:09 +0100)
Signed-off-by: Thomas Zimmermann <tdz@users.sourceforge.net>
[smcv: add space after cast, that is our coding style]
Reviewed-by: Simon McVittie <smcv@debian.org>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=97357

dbus/dbus-sysdeps-win.c

index bd18ddb..f340708 100644 (file)
@@ -631,12 +631,12 @@ _dbus_write_socket_two (DBusSocket        fd,
     {
       DBUS_SOCKET_SET_ERRNO ();
       _dbus_verbose ("WSASend: failed: %s\n", _dbus_strerror_from_errno ());
-      bytes_written = -1;
+      bytes_written = (DWORD) -1;
     }
   else
     _dbus_verbose ("WSASend: = %ld\n", bytes_written);
     
-  if (bytes_written < 0 && errno == EINTR)
+  if (bytes_written == (DWORD) -1 && errno == EINTR)
     goto again;
       
   return bytes_written;