[lmi-commits] [5835] Make "Census | Print case to disk" cancellable

[Greg Chicares]

Revision: 5835
          http://svn.sv.gnu.org/viewvc/?view=rev&root=lmi&revision=5835
Author:   chicares
Date:     2013-11-17 14:03:14 +0000 (Sun, 17 Nov 2013)
Log Message:
-----------
Make "Census | Print case to disk" cancellable

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

Modified: lmi/trunk/ChangeLog
===================================================================
--- lmi/trunk/ChangeLog 2013-11-17 12:11:29 UTC (rev 5834)
+++ lmi/trunk/ChangeLog 2013-11-17 14:03:14 UTC (rev 5835)
@@ -32939,3 +32939,9 @@
 Abandon the 20131114T1431Z experiment. See:
   http://lists.nongnu.org/archive/html/lmi/2013-11/msg00015.html
 
+20131117T1403Z <address@hidden> [542]
+
+  system_command_wx.cpp
+Make "Census | Print case to disk" cancellable. See:
+  http://lists.nongnu.org/archive/html/lmi/2013-11/msg00017.html
+

Modified: lmi/trunk/system_command_wx.cpp
===================================================================
--- lmi/trunk/system_command_wx.cpp     2013-11-17 12:11:29 UTC (rev 5834)
+++ lmi/trunk/system_command_wx.cpp     2013-11-17 14:03:14 UTC (rev 5835)
@@ -74,6 +74,11 @@
 /// exit code and the command itself.
 ///
 /// Show elapsed time on statusbar iff statusbar is available.
+///
+/// See:
+///   http://lists.nongnu.org/archive/html/lmi/2013-11/msg00017.html
+/// for the wxEXEC_NODISABLE rationale. This is potentially dangerous,
+/// and could be inhibited (by an extra argument) if ever needed.
 
 void concrete_system_command(std::string const& command_line)
 {
@@ -90,7 +95,7 @@
     statusbar_if_available << "Running..." << std::flush;
     wxArrayString output;
     wxArrayString errors;
-    long int exit_code = wxExecute(command_line, output, errors);
+    long int exit_code = wxExecute(command_line, output, errors, 
wxEXEC_NODISABLE);
     statusbar_if_available << timer.stop().elapsed_msec_str() << std::flush;
 
     if(0L == exit_code)