fix wxSocket notifications under MSW after the last change

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

diff --git a/include/wx/gsocket.h b/include/wx/gsocket.h
index 4d1c6215965b0bdcc7ea09a8d282094d712a831a..d2fc0d07dd079738f276f66610964fe727780e81 100644 (file)
--- a/include/wx/gsocket.h
+++ b/include/wx/gsocket.h
@@ -179,6 +179,10 @@ public:
     virtual void Close() = 0;
     virtual void Shutdown();
 
     virtual void Close() = 0;
     virtual void Shutdown();
 

+    // notify m_wxsocket about the given socket event by calling its (inaptly
+    // named) OnRequest() method
+    void NotifyOnStateChange(GSocketEvent event);
+

     // this is officially SOCKET (unsigned int) under Windows but we don't want
     // to include winsock.h which defines SOCKET from here so just use int
     // under all platforms
     // this is officially SOCKET (unsigned int) under Windows but we don't want
     // to include winsock.h which defines SOCKET from here so just use int
     // under all platforms


@@ -210,9 +214,6 @@ public:
 protected:
     GSocketBase(wxSocketBase& wxsocket);
 
 protected:
     GSocketBase(wxSocketBase& wxsocket);
 

-    // notify m_wxsocket
-    void NotifyOnStateChange(GSocketEvent event);
-

 private:
     // set in ctor and never changed except that it's reset to NULL when the
     // socket is shut down
 private:
     // set in ctor and never changed except that it's reset to NULL when the
     // socket is shut down

diff --git a/src/msw/gsockmsw.cpp b/src/msw/gsockmsw.cpp
index 3ca8b48349e4ae395c8d6f798741eca4b3bbdc03..1975b983a058b3a3dc3f0a99b5bd1dd79b697770 100644 (file)
--- a/src/msw/gsockmsw.cpp
+++ b/src/msw/gsockmsw.cpp
@@ -387,9 +387,6 @@ LRESULT CALLBACK _GSocket_Internal_WinProc(HWND hWnd,
 
       if (event != -1)
       {
 
       if (event != -1)
       {

-        cback = socket->m_cbacks[event];
-        data = socket->m_data[event];
-

         if (event == GSOCK_LOST)
           socket->m_detected = GSOCK_LOST_FLAG;
         else
         if (event == GSOCK_LOST)
           socket->m_detected = GSOCK_LOST_FLAG;
         else


@@ -397,16 +394,10 @@ LRESULT CALLBACK _GSocket_Internal_WinProc(HWND hWnd,
       }
     }
 
       }
     }
 

-    /* OK, we can now leave the critical section because we have
-     * already obtained the callback address (we make no further
-     * accesses to socket->whatever). However, the app should
-     * be prepared to handle events from a socket that has just
-     * been closed!
-     */

     LeaveCriticalSection(&critical);
 
     LeaveCriticalSection(&critical);
 

-    if (cback != NULL)
-      (cback)(socket, event, data);
+    if ( socket )
+        socket->NotifyOnStateChange(event);

 
     return (LRESULT) 0;
   }
 
     return (LRESULT) 0;
   }