DBusSocket: put the #ifdef case before the !defined case

This avoids the confusing #ifndef...#else anti-pattern.

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=89444
Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de>

diff --git a/dbus/dbus-sysdeps.h b/dbus/dbus-sysdeps.h
index 615950b77cee552011a7cc786aaa589f8d31dd23..f6a2948ec19ddb745b9b19d17a8443d65d478643 100644 (file)
--- a/dbus/dbus-sysdeps.h
+++ b/dbus/dbus-sysdeps.h
@@ -125,43 +125,43 @@ typedef unsigned long dbus_gid_t;
 /**
  * Socket interface
  */
-#ifndef DBUS_WIN
+#ifdef DBUS_WIN
 
-typedef struct { int fd; } DBusSocket;
-# define DBUS_SOCKET_FORMAT "d"
-# define DBUS_SOCKET_INIT { -1 }
+typedef struct { SOCKET sock; } DBusSocket;
+# define DBUS_SOCKET_FORMAT "Iu"
+# define DBUS_SOCKET_INIT { INVALID_SOCKET }
 
-static inline int
-_dbus_socket_printable (DBusSocket s) { return s.fd; }
+static inline SOCKET
+_dbus_socket_printable (DBusSocket s) { return s.sock; }
 
 static inline dbus_bool_t
-_dbus_socket_is_valid (DBusSocket s) { return s.fd >= 0; }
+_dbus_socket_is_valid (DBusSocket s) { return s.sock != INVALID_SOCKET; }
 
 static inline void
-_dbus_socket_invalidate (DBusSocket *s) { s->fd = -1; }
+_dbus_socket_invalidate (DBusSocket *s) { s->sock = INVALID_SOCKET; }
 
 static inline int
-_dbus_socket_get_int (DBusSocket s) { return s.fd; }
+_dbus_socket_get_int (DBusSocket s) { return (int)s.sock; }
 
-#else /* DBUS_WIN */
+#else /* not DBUS_WIN */
 
-typedef struct { SOCKET sock; } DBusSocket;
-# define DBUS_SOCKET_FORMAT "Iu"
-# define DBUS_SOCKET_INIT { INVALID_SOCKET }
+typedef struct { int fd; } DBusSocket;
+# define DBUS_SOCKET_FORMAT "d"
+# define DBUS_SOCKET_INIT { -1 }
 
-static inline SOCKET
-_dbus_socket_printable (DBusSocket s) { return s.sock; }
+static inline int
+_dbus_socket_printable (DBusSocket s) { return s.fd; }
 
 static inline dbus_bool_t
-_dbus_socket_is_valid (DBusSocket s) { return s.sock != INVALID_SOCKET; }
+_dbus_socket_is_valid (DBusSocket s) { return s.fd >= 0; }
 
 static inline void
-_dbus_socket_invalidate (DBusSocket *s) { s->sock = INVALID_SOCKET; }
+_dbus_socket_invalidate (DBusSocket *s) { s->fd = -1; }
 
 static inline int
-_dbus_socket_get_int (DBusSocket s) { return (int)s.sock; }
+_dbus_socket_get_int (DBusSocket s) { return s.fd; }
 
-#endif /* DBUS_WIN */
+#endif /* not DBUS_WIN */
 
 static inline DBusSocket
 _dbus_socket_get_invalid (void)