Changed code to get rid of a compiler warning about statement always being FALSE

[wxWidgets.git] / src / gtk / gsockgtk.c

diff --git a/src/gtk/gsockgtk.c b/src/gtk/gsockgtk.c
index e462a6aecea7c2acd872ba186e5dbea2a8e65f1a..8551a7d13b08fa921aa85563829e64fc2e690be4 100644 (file)
--- a/src/gtk/gsockgtk.c
+++ b/src/gtk/gsockgtk.c
@@ -10,89 +10,83 @@
 #if wxUSE_SOCKETS
 
 #include <stdlib.h>
+#include <stdio.h>
+
 #include <gdk/gdk.h>
 #include <glib.h>
-#include <wx/gsocket.h>
-#include "../unix/gsockunx.h"
 
-void _GSocket_GDK_Input(gpointer data, gint source, GdkInputCondition condition)
+#include "wx/gsocket.h"
+#include "wx/unix/gsockunx.h"
+
+
+void _GSocket_GDK_Input(gpointer data,
+                        gint source,
+                        GdkInputCondition condition)
 {
   GSocket *socket = (GSocket *)data;
 
-  switch (condition) {
-  case GDK_INPUT_READ:
+  if (condition & GDK_INPUT_READ)
     _GSocket_Detected_Read(socket);
-    break;
-  case GDK_INPUT_WRITE:
+  if (condition & GDK_INPUT_WRITE)
     _GSocket_Detected_Write(socket);
-    break;
-  default:
-    break;
-  } 
 }
 
-void _GSocket_GUI_Init(GSocket *socket)
+bool _GSocket_GUI_Init(GSocket *socket)
 {
-  int i;
   gint *m_id;
 
-  socket->m_gui_dependent = (char *)malloc(sizeof(gint)*3);
+  socket->m_gui_dependent = (char *)malloc(sizeof(gint)*2);
   m_id = (gint *)(socket->m_gui_dependent);
 
-  for (i=0;i<3;i++)
-    m_id[i] = -1;
+  m_id[0] = -1;
+  m_id[1] = -1;
+
+  return TRUE;
 }
 
 void _GSocket_GUI_Destroy(GSocket *socket)
 {
-  int i;
-  gint *m_id;
-
-  m_id = (gint *)(socket->m_gui_dependent);
-
-  for (i=0;i<3;i++)
-    if (m_id[i] == -1)
-      gdk_input_remove(m_id[i]);
-
   free(socket->m_gui_dependent);
 }
 
 void _GSocket_Install_Callback(GSocket *socket, GSocketEvent event)
 {
-  GdkInputCondition flag;
+  gint *m_id = (gint *)(socket->m_gui_dependent);
   int c;
-  gint *m_id;
 
-  m_id = (gint *)(socket->m_gui_dependent);
+  if (socket->m_fd == -1)
+    return;
 
-  switch (event) {
-  case GSOCK_CONNECTION:
-  case GSOCK_LOST:
-  case GSOCK_INPUT: c = 0; flag = GDK_INPUT_READ; break;
-  case GSOCK_OUTPUT: c = 1;flag = GDK_INPUT_WRITE; break;
-  default: return;
+  switch (event)
+  {
+    case GSOCK_LOST:       /* fall-through */
+    case GSOCK_INPUT:      c = 0; break;
+    case GSOCK_OUTPUT:     c = 1; break;
+    case GSOCK_CONNECTION: c = ((socket->m_server) ? 0 : 1); break;
+    default: return;
   }
 
   if (m_id[c] != -1)
     gdk_input_remove(m_id[c]);
 
-  m_id[c] = gdk_input_add(socket->m_fd, flag, 
-                          _GSocket_GDK_Input, (gpointer)socket);
+  m_id[c] = gdk_input_add(socket->m_fd,
+                          (c ? GDK_INPUT_WRITE : GDK_INPUT_READ),
+                          _GSocket_GDK_Input,
+                          (gpointer)socket);
 }
 
 void _GSocket_Uninstall_Callback(GSocket *socket, GSocketEvent event)
 {
+  gint *m_id = (gint *)(socket->m_gui_dependent);
   int c;
-  gint *m_id;
 
-  m_id = (gint *)(socket->m_gui_dependent);
-
-  switch (event) {
-  case GSOCK_CONNECTION: 
-  case GSOCK_LOST:
-  case GSOCK_INPUT: c = 0; break;
-  case GSOCK_OUTPUT: c = 1; break;
-  default: return;
+  switch (event)
+  {
+    case GSOCK_LOST:       /* fall-through */
+    case GSOCK_INPUT:      c = 0; break;
+    case GSOCK_OUTPUT:     c = 1; break;
+    case GSOCK_CONNECTION: c = ((socket->m_server) ? 0 : 1); break;
+    default: return;
   }
 
   if (m_id[c] != -1)
@@ -101,13 +95,21 @@ void _GSocket_Uninstall_Callback(GSocket *socket, GSocketEvent event)
   m_id[c] = -1;
 }
 
-unsigned long GSocket_GetEventID(GSocket *socket)
+void _GSocket_Enable_Events(GSocket *socket)
 {
-  return 0;
+  _GSocket_Install_Callback(socket, GSOCK_INPUT);
+  _GSocket_Install_Callback(socket, GSOCK_OUTPUT);
 }
 
-void GSocket_DoEvent(unsigned long evt_id)
+void _GSocket_Disable_Events(GSocket *socket)
 {
+  _GSocket_Uninstall_Callback(socket, GSOCK_INPUT);
+  _GSocket_Uninstall_Callback(socket, GSOCK_OUTPUT);
 }
 
-#endif /* wxUSE_SOCKETS */
+#else /* !wxUSE_SOCKETS */
+
+/* some compilers don't like having empty source files */
+static int wxDummyGsockVar = 0;
+
+#endif /* wxUSE_SOCKETS/!wxUSE_SOCKETS */