[lmi-commits] [5834] Abandon the 20131114T1431Z experiment

[Greg Chicares]

Revision: 5834
          http://svn.sv.gnu.org/viewvc/?view=rev&root=lmi&revision=5834
Author:   chicares
Date:     2013-11-17 12:11:29 +0000 (Sun, 17 Nov 2013)
Log Message:
-----------
Abandon the 20131114T1431Z experiment

Modified Paths:
--------------
    lmi/trunk/ChangeLog
    lmi/trunk/system_command_wx.cpp

Modified: lmi/trunk/ChangeLog
===================================================================
--- lmi/trunk/ChangeLog 2013-11-15 01:03:15 UTC (rev 5833)
+++ lmi/trunk/ChangeLog 2013-11-17 12:11:29 UTC (rev 5834)
@@ -32933,3 +32933,9 @@
   install_msw.sh
 Make pause between printouts configurable.
 
+20131117T1211Z <address@hidden> [542]
+
+  system_command_wx.cpp
+Abandon the 20131114T1431Z experiment. See:
+  http://lists.nongnu.org/archive/html/lmi/2013-11/msg00015.html
+

Modified: lmi/trunk/system_command_wx.cpp
===================================================================
--- lmi/trunk/system_command_wx.cpp     2013-11-15 01:03:15 UTC (rev 5833)
+++ lmi/trunk/system_command_wx.cpp     2013-11-17 12:11:29 UTC (rev 5834)
@@ -33,7 +33,6 @@
 #include "timer.hpp"
 
 #include <wx/app.h>                     // wxTheApp
-#include <wx/evtloop.h>                 // class wxEventLoopBase
 #include <wx/frame.h>
 #include <wx/utils.h>
 
@@ -75,11 +74,6 @@
 /// exit code and the command itself.
 ///
 /// Show elapsed time on statusbar iff statusbar is available.
-///
-/// Before calling wxExecute(), yield for UI events so that Cancel is
-/// recognized when this function is called from a wxProgressDialog
-/// loop. See the discussion thread starting here:
-///   http://lists.nongnu.org/archive/html/lmi/2013-11/msg00009.html
 
 void concrete_system_command(std::string const& command_line)
 {
@@ -92,11 +86,8 @@
         && dynamic_cast<wxFrame*>(wxTheApp->GetTopWindow())->GetStatusBar()
         ;
     std::ostream& statusbar_if_available = b ? status() : null_stream();
-    statusbar_if_available << "Running..." << std::flush;
 
-    wxEventLoopBase* e = wxEventLoopBase::GetActive();
-    e && e->YieldFor(wxEVT_CATEGORY_UI);
-
+    statusbar_if_available << "Running..." << std::flush;
     wxArrayString output;
     wxArrayString errors;
     long int exit_code = wxExecute(command_line, output, errors);