use a virtual function instead of wxDynamicCast(wxMDIParentFrame) in wxFrame code...

[wxWidgets.git] / include / wx / unix / private / sockunix.h

diff --git a/include/wx/unix/private/sockunix.h b/include/wx/unix/private/sockunix.h
index 472d524a565e78b5b26b41c011d6a55f7d052bf4..25310603649f7f44f8d41c435b5addec868fb81e 100644 (file)
--- a/include/wx/unix/private/sockunix.h
+++ b/include/wx/unix/private/sockunix.h
@@ -14,8 +14,10 @@
 
 #include <unistd.h>
 #include <sys/ioctl.h>
 
 #include <unistd.h>
 #include <sys/ioctl.h>

+#include "wx/private/fdiodispatcher.h"

 
 

-class wxSocketImplUnix : public wxSocketImpl
+class wxSocketImplUnix : public wxSocketImpl,
+                         public wxFDIOHandler

 {
 public:
     wxSocketImplUnix(wxSocketBase& wxsocket)
 {
 public:
     wxSocketImplUnix(wxSocketBase& wxsocket)


@@ -24,29 +26,45 @@ public:
         m_fds[0] =
         m_fds[1] = -1;
 
         m_fds[0] =
         m_fds[1] = -1;
 

-        m_use_events = false;
+        m_enabledCallbacks = 0;

     }
 
     }
 

-    virtual void Shutdown();
-    virtual wxSocketImpl *WaitConnection(wxSocketBase& wxsocket);
+    virtual wxSocketError GetLastError() const;

 
 

-    int Read(char *buffer, int size);
-    int Write(const char *buffer, int size);
-    //attach or detach from main loop
-    void Notify(bool flag);
-    void Detected_Read();
-    void Detected_Write();
+    virtual void ReenableEvents(wxSocketEventFlags flags)
+    {
+        // enable the notifications about input/output being available again in
+        // case they were disabled by OnRead/WriteWaiting()
+        //
+        // notice that we'd like to enable the events here only if there is
+        // nothing more left on the socket right now as otherwise we're going
+        // to get a "ready for whatever" notification immediately (well, during
+        // the next event loop iteration) and disable the event back again
+        // which is rather inefficient but unfortunately doing it like this
+        // doesn't work because the existing code (e.g. src/common/sckipc.cpp)
+        // expects to keep getting notifications about the data available from
+        // the socket even if it didn't read all the data the last time, so we
+        // absolutely have to continue generating them
+        EnableEvents(flags);
+    }
+
+    // wxFDIOHandler methods
+    virtual void OnReadWaiting();
+    virtual void OnWriteWaiting();
+    virtual void OnExceptionWaiting();
+
+    // Unix-specific functions used by wxSocketFDIOManager only
+    bool HasAnyEnabledCallbacks() const { return m_enabledCallbacks != 0; }
+    void EnableCallback(wxFDIODispatcherEntryFlags flag)
+        { m_enabledCallbacks |= flag; }
+    void DisableCallback(wxFDIODispatcherEntryFlags flag)
+        { m_enabledCallbacks &= ~flag; }
+    int GetEnabledCallbacks() const { return m_enabledCallbacks; }

 
 private:
 
 private:

-    virtual wxSocketError DoHandleConnect(int ret);

     virtual void DoClose()
     {
     virtual void DoClose()
     {

-        wxSocketManager * const manager = wxSocketManager::Get();
-        if ( manager )
-        {
-            manager->Uninstall_Callback(this, wxSOCKET_INPUT);
-            manager->Uninstall_Callback(this, wxSOCKET_OUTPUT);
-        }
+        DisableEvents();

 
         close(m_fd);
     }
 
         close(m_fd);
     }


@@ -59,54 +77,34 @@ private:
         EnableEvents();
     }
 
         EnableEvents();
     }
 

-    // enable or disable notifications for socket input/output events but only
-    // if m_use_events is true; do nothing otherwise
-    virtual void EnableEvents()
-    {
-        if ( m_use_events )
-            DoEnableEvents(true);
-    }
-
-    void DisableEvents()
-    {
-        if ( m_use_events )
-            DoEnableEvents(false);
-    }
+    // enable or disable notifications for socket input/output events
+    void EnableEvents(int flags = wxSOCKET_INPUT_FLAG | wxSOCKET_OUTPUT_FLAG)
+        { DoEnableEvents(flags, true); }
+    void DisableEvents(int flags = wxSOCKET_INPUT_FLAG | wxSOCKET_OUTPUT_FLAG)
+        { DoEnableEvents(flags, false); }

 
 

-    // really enable or disable socket input/output events, regardless of
-    // m_use_events value
-    void DoEnableEvents(bool enable);
-
-
-    // enable or disable events for the given event if m_use_events; do nothing
-    // otherwise
-    //
-    // notice that these functions also update m_detected: EnableEvent() clears
-    // the corresponding bit in it and DisableEvent() sets it
-    void EnableEvent(wxSocketNotify event);
-    void DisableEvent(wxSocketNotify event);
-
-
-    wxSocketError Input_Timeout();
-    wxSocketError Output_Timeout();
-    int Recv_Stream(char *buffer, int size);
-    int Recv_Dgram(char *buffer, int size);
-    int Send_Stream(const char *buffer, int size);
-    int Send_Dgram(const char *buffer, int size);
+    // really enable or disable socket input/output events
+    void DoEnableEvents(int flags, bool enable);

 
 protected:
 
 protected:

-    // true if socket should fire events
-    bool m_use_events;
-

     // descriptors for input and output event notification channels associated
     // with the socket
     int m_fds[2];
 
     // descriptors for input and output event notification channels associated
     // with the socket
     int m_fds[2];
 

+    // the events which are currently enabled for this socket, combination of
+    // wxFDIO_INPUT and wxFDIO_OUTPUT values
+    int m_enabledCallbacks;
+

 private:
     // notify the associated wxSocket about a change in socket state and shut
     // down the socket if the event is wxSOCKET_LOST
     void OnStateChange(wxSocketNotify event);
 
 private:
     // notify the associated wxSocket about a change in socket state and shut
     // down the socket if the event is wxSOCKET_LOST
     void OnStateChange(wxSocketNotify event);
 

+    // check if there is any input available, return 1 if yes, 0 if no or -1 on
+    // error
+    int CheckForInput();
+
+

     // give it access to our m_fds
     friend class wxSocketFDBasedManager;
 };
     // give it access to our m_fds
     friend class wxSocketFDBasedManager;
 };


@@ -119,12 +117,6 @@ public:
     virtual bool OnInit() { return true; }
     virtual void OnExit() { }
 
     virtual bool OnInit() { return true; }
     virtual void OnExit() { }
 

-    // allocate/free the storage we need
-    virtual wxSocketImpl *CreateSocket(wxSocketBase& wxsocket)
-    {
-        return new wxSocketImplUnix(wxsocket);
-    }
-

 protected:
     // identifies either input or output direction
     //
 protected:
     // identifies either input or output direction
     //


@@ -141,11 +133,12 @@ protected:
         switch ( event )
         {
             default:
         switch ( event )
         {
             default:

-                wxFAIL_MSG( "unexpected socket event" );
-                // fall through
+                wxFAIL_MSG( "unknown socket event" );
+                return FD_INPUT; // we must return something

 
             case wxSOCKET_LOST:
 
             case wxSOCKET_LOST:

-                // fall through
+                wxFAIL_MSG( "unexpected socket event" );
+                return FD_INPUT; // as above

 
             case wxSOCKET_INPUT:
                 return FD_INPUT;
 
             case wxSOCKET_INPUT:
                 return FD_INPUT;


@@ -154,15 +147,20 @@ protected:
                 return FD_OUTPUT;
 
             case wxSOCKET_CONNECTION:
                 return FD_OUTPUT;
 
             case wxSOCKET_CONNECTION:

-                // FIXME: explain this?
-                return socket->m_server ? FD_INPUT : FD_OUTPUT;
+                // for server sockets we're interested in events indicating
+                // that a new connection is pending, i.e. that accept() will
+                // succeed and this is indicated by socket becoming ready for
+                // reading, while for the other ones we're interested in the
+                // completion of non-blocking connect() which is indicated by
+                // the socket becoming ready for writing
+                return socket->IsServer() ? FD_INPUT : FD_OUTPUT;

         }
     }
 
     // access the FDs we store
         }
     }
 
     // access the FDs we store

-    int& FD(wxSocketImpl *socket, SocketDir d)
+    int& FD(wxSocketImplUnix *socket, SocketDir d)

     {
     {

-        return static_cast<wxSocketImplUnix *>(socket)->m_fds[d];
+        return socket->m_fds[d];

     }
 };
 
     }
 };
 


@@ -171,8 +169,11 @@ protected:
 class wxSocketInputBasedManager : public wxSocketFDBasedManager
 {
 public:
 class wxSocketInputBasedManager : public wxSocketFDBasedManager
 {
 public:

-    virtual void Install_Callback(wxSocketImpl *socket, wxSocketNotify event)
+    virtual void Install_Callback(wxSocketImpl *socket_, wxSocketNotify event)

     {
     {

+        wxSocketImplUnix * const
+            socket = static_cast<wxSocketImplUnix *>(socket_);
+

         wxCHECK_RET( socket->m_fd != -1,
                         "shouldn't be called on invalid socket" );
 
         wxCHECK_RET( socket->m_fd != -1,
                         "shouldn't be called on invalid socket" );
 


@@ -182,11 +183,14 @@ public:
         if ( fd != -1 )
             RemoveInput(fd);
 
         if ( fd != -1 )
             RemoveInput(fd);
 

-        fd = AddInput(socket, d);
+        fd = AddInput(socket, socket->m_fd, d);

     }
 
     }
 

-    virtual void Uninstall_Callback(wxSocketImpl *socket, wxSocketNotify event)
+    virtual void Uninstall_Callback(wxSocketImpl *socket_, wxSocketNotify event)

     {
     {

+        wxSocketImplUnix * const
+            socket = static_cast<wxSocketImplUnix *>(socket_);
+

         const SocketDir d = GetDirForEvent(socket, event);
 
         int& fd = FD(socket, d);
         const SocketDir d = GetDirForEvent(socket, event);
 
         int& fd = FD(socket, d);


@@ -200,7 +204,7 @@ public:
 private:
     // these functions map directly to XtAdd/RemoveInput() or
     // gdk_input_add/remove()
 private:
     // these functions map directly to XtAdd/RemoveInput() or
     // gdk_input_add/remove()

-    virtual int AddInput(wxSocketImpl *socket, SocketDir d) = 0;
+    virtual int AddInput(wxFDIOHandler *handler, int fd, SocketDir d) = 0;

     virtual void RemoveInput(int fd) = 0;
 };
 
     virtual void RemoveInput(int fd) = 0;
 };