call ProcessPendingEvents() from wxAppConsoleBase::ProcessIdle() too, not only from...

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

diff --git a/include/wx/msw/evtloop.h b/include/wx/msw/evtloop.h
index 88ab13492bfe1d0b71b0560908344534ca8a359e..2869b07a92449a77532295830e0a4ec3ea83e468 100644 (file)
--- a/include/wx/msw/evtloop.h
+++ b/include/wx/msw/evtloop.h
@@ -102,9 +102,6 @@ public:
 
     // MSW-specific function to process a single message
     virtual void ProcessMessage(WXMSG *msg);
-
-protected:
-    virtual void OnNextIteration();
 };
 
 #endif // wxUSE_CONSOLE_EVENTLOOP

diff --git a/src/common/appbase.cpp b/src/common/appbase.cpp
index a02e091ff1575b84454f4997b9b9dff2f473cd7a..b2a0fd718f7df4e5d2ab46050c2cdf0400f4e268 100644 (file)
--- a/src/common/appbase.cpp
+++ b/src/common/appbase.cpp
@@ -385,6 +385,9 @@ void wxAppConsoleBase::WakeUpIdle()
 
 bool wxAppConsoleBase::ProcessIdle()
 {
+    // process pending wx events before sending idle events
+    ProcessPendingEvents();
+
     wxIdleEvent event;
 
     event.SetEventObject(this);

diff --git a/src/common/appcmn.cpp b/src/common/appcmn.cpp
index a38f84583f1ce5dbcb9273c8e10b1cfd98da94fb..af5a1d8eceed2902caa0b532b4c7644760cfe43f 100644 (file)
--- a/src/common/appcmn.cpp
+++ b/src/common/appcmn.cpp
@@ -353,11 +353,11 @@ void wxAppBase::DeletePendingObjects()
 // Returns true if more time is needed.
 bool wxAppBase::ProcessIdle()
 {
-    // process pending wx events before sending idle events
-    ProcessPendingEvents();
-
+    // call the base class version first, it will process the pending events
+    // (which should be done before the idle events generation) and send the
+    // idle event to wxTheApp itself
+    bool needMore = wxAppConsoleBase::ProcessIdle();
     wxIdleEvent event;
-    bool needMore = false;
     wxWindowList::compatibility_iterator node = wxTopLevelWindows.GetFirst();
     while (node)
     {
@@ -367,9 +367,6 @@ bool wxAppBase::ProcessIdle()
         node = node->GetNext();
     }
 
-    if (wxAppConsole::ProcessIdle())
-        needMore = true;
-
     // 'Garbage' collection of windows deleted with Close().
     DeletePendingObjects();
 

diff --git a/src/msw/evtloop.cpp b/src/msw/evtloop.cpp
index 0b5530ff70b78b5fa96ebb9a9b04c5870c65e471..d09faae69f8a00b25b74cac953c4ed126f114c9c 100644 (file)
--- a/src/msw/evtloop.cpp
+++ b/src/msw/evtloop.cpp
@@ -363,12 +363,6 @@ void wxGUIEventLoop::WakeUp()
 
 #if wxUSE_CONSOLE_EVENTLOOP
 
-void wxConsoleEventLoop::OnNextIteration()
-{
-    if ( wxTheApp )
-        wxTheApp->ProcessPendingEvents();
-}
-
 void wxConsoleEventLoop::WakeUp()
 {
 #if wxUSE_THREADS

diff --git a/src/unix/evtloopunix.cpp b/src/unix/evtloopunix.cpp
index 7f27c1c2a216acba8033c177be65510ad074e52d..1d5c1dc1a080f8f5d605788c1a373b8afb460020 100644 (file)
--- a/src/unix/evtloopunix.cpp
+++ b/src/unix/evtloopunix.cpp
@@ -174,8 +174,6 @@ int wxConsoleEventLoop::DispatchTimeout(unsigned long timeout)
         hadEvent = true;
 #endif // wxUSE_TIMER
 
-    wxTheApp->ProcessPendingEvents();
-
     return hadEvent ? 1 : -1;
 }