]> git.saurik.com Git - wxWidgets.git/blobdiff - samples/exec/exec.cpp
Don't use GetThreadId() in wxMSW code.
[wxWidgets.git] / samples / exec / exec.cpp
index c265104c7bc945be3470999e94c9f47a044e48dd..d72c89a7adb37a9c9c81de81ae4685e16fd44da9 100644 (file)
 // headers
 // ----------------------------------------------------------------------------
 
 // headers
 // ----------------------------------------------------------------------------
 
-#if defined(__GNUG__) && !defined(__APPLE__)
-    #pragma implementation "exec.cpp"
-    #pragma interface "exec.cpp"
-#endif
-
 // For compilers that support precompilation, includes "wx/wx.h".
 #include "wx/wxprec.h"
 
 // For compilers that support precompilation, includes "wx/wx.h".
 #include "wx/wxprec.h"
 
@@ -30,7 +25,7 @@
 #endif
 
 // for all others, include the necessary headers (this file is usually all you
 #endif
 
 // for all others, include the necessary headers (this file is usually all you
-// need because it includes almost all "standard" wxWindows headers
+// need because it includes almost all "standard" wxWidgets headers
 #ifndef WX_PRECOMP
     #include "wx/app.h"
     #include "wx/log.h"
 #ifndef WX_PRECOMP
     #include "wx/app.h"
     #include "wx/log.h"
@@ -48,6 +43,8 @@
     #include "wx/choicdlg.h"
 
     #include "wx/button.h"
     #include "wx/choicdlg.h"
 
     #include "wx/button.h"
+    #include "wx/checkbox.h"
+    #include "wx/stattext.h"
     #include "wx/textctrl.h"
     #include "wx/listbox.h"
 
     #include "wx/textctrl.h"
     #include "wx/listbox.h"
 
 #endif
 
 #include "wx/txtstrm.h"
 #endif
 
 #include "wx/txtstrm.h"
+#include "wx/numdlg.h"
+#include "wx/textdlg.h"
+#include "wx/ffile.h"
+#include "wx/scopedptr.h"
+#include "wx/stopwatch.h"
 
 #include "wx/process.h"
 
 
 #include "wx/process.h"
 
     #include "wx/dde.h"
 #endif // __WINDOWS__
 
     #include "wx/dde.h"
 #endif // __WINDOWS__
 
+#ifndef wxHAS_IMAGES_IN_RESOURCES
+    #include "../sample.xpm"
+#endif
+
 // ----------------------------------------------------------------------------
 // the usual application and main frame classes
 // ----------------------------------------------------------------------------
 // ----------------------------------------------------------------------------
 // the usual application and main frame classes
 // ----------------------------------------------------------------------------
@@ -83,14 +89,18 @@ public:
 
 // Define an array of process pointers used by MyFrame
 class MyPipedProcess;
 
 // Define an array of process pointers used by MyFrame
 class MyPipedProcess;
-WX_DEFINE_ARRAY_NO_PTR(MyPipedProcess *, MyProcessesArray);
+WX_DEFINE_ARRAY_PTR(MyPipedProcess *, MyPipedProcessesArray);
+
+class MyProcess;
+WX_DEFINE_ARRAY_PTR(MyProcess *, MyProcessesArray);
 
 // Define a new frame type: this is going to be our main frame
 class MyFrame : public wxFrame
 {
 public:
 
 // Define a new frame type: this is going to be our main frame
 class MyFrame : public wxFrame
 {
 public:
-    // ctor(s)
+    // ctor and dtor
     MyFrame(const wxString& title, const wxPoint& pos, const wxSize& size);
     MyFrame(const wxString& title, const wxPoint& pos, const wxSize& size);
+    virtual ~MyFrame();
 
     // event handlers (these functions should _not_ be virtual)
     void OnQuit(wxCommandEvent& event);
 
     // event handlers (these functions should _not_ be virtual)
     void OnQuit(wxCommandEvent& event);
@@ -99,6 +109,9 @@ public:
 
     void OnClear(wxCommandEvent& event);
 
 
     void OnClear(wxCommandEvent& event);
 
+    void OnBeginBusyCursor(wxCommandEvent& event);
+    void OnEndBusyCursor(wxCommandEvent& event);
+
     void OnSyncExec(wxCommandEvent& event);
     void OnAsyncExec(wxCommandEvent& event);
     void OnShell(wxCommandEvent& event);
     void OnSyncExec(wxCommandEvent& event);
     void OnAsyncExec(wxCommandEvent& event);
     void OnShell(wxCommandEvent& event);
@@ -108,48 +121,40 @@ public:
     void OnPOpen(wxCommandEvent& event);
 
     void OnFileExec(wxCommandEvent& event);
     void OnPOpen(wxCommandEvent& event);
 
     void OnFileExec(wxCommandEvent& event);
+    void OnFileLaunch(wxCommandEvent& event);
+    void OnOpenURL(wxCommandEvent& event);
+    void OnShowCommandForExt(wxCommandEvent& event);
 
     void OnAbout(wxCommandEvent& event);
 
     // polling output of async processes
 
     void OnAbout(wxCommandEvent& event);
 
     // polling output of async processes
-    void OnTimer(wxTimerEvent& event);
+    void OnIdleTimer(wxTimerEvent& event);
     void OnIdle(wxIdleEvent& event);
 
     // for MyPipedProcess
     void OnProcessTerminated(MyPipedProcess *process);
     wxListBox *GetLogListBox() const { return m_lbox; }
 
     void OnIdle(wxIdleEvent& event);
 
     // for MyPipedProcess
     void OnProcessTerminated(MyPipedProcess *process);
     wxListBox *GetLogListBox() const { return m_lbox; }
 
+    // for MyProcess
+    void OnAsyncTermination(MyProcess *process);
+
+    // timer updating a counter in the background
+    void OnBgTimer(wxTimerEvent& event);
+
 private:
     void ShowOutput(const wxString& cmd,
                     const wxArrayString& output,
                     const wxString& title);
 
 private:
     void ShowOutput(const wxString& cmd,
                     const wxArrayString& output,
                     const wxString& title);
 
-    void DoAsyncExec(const wxString& cmd);
+    int GetExecFlags() const;
 
 
-    void AddAsyncProcess(MyPipedProcess *process)
-    {
-        if ( m_running.IsEmpty() )
-        {
-            // we want to start getting the timer events to ensure that a
-            // steady stream of idle events comes in -- otherwise we
-            // wouldn't be able to poll the child process input
-            m_timerIdleWakeUp.Start(100);
-        }
-        //else: the timer is already running
+    void DoAsyncExec(const wxString& cmd);
 
 
-        m_running.Add(process);
-    }
+    void AddAsyncProcess(MyProcess *process) { m_allAsync.push_back(process); }
 
 
-    void RemoveAsyncProcess(MyPipedProcess *process)
-    {
-        m_running.Remove(process);
+    void AddPipedProcess(MyPipedProcess *process);
+    void RemovePipedProcess(MyPipedProcess *process);
 
 
-        if ( m_running.IsEmpty() )
-        {
-            // we don't need to get idle events all the time any more
-            m_timerIdleWakeUp.Stop();
-        }
-    }
 
     // the PID of the last process we launched asynchronously
     long m_pidLast;
 
     // the PID of the last process we launched asynchronously
     long m_pidLast;
@@ -171,12 +176,20 @@ private:
 
     wxListBox *m_lbox;
 
 
     wxListBox *m_lbox;
 
-    MyProcessesArray m_running;
+    // array of running processes with redirected IO
+    MyPipedProcessesArray m_running;
+
+    // array of all asynchrously running processes
+    MyProcessesArray m_allAsync;
 
     // the idle event wake up timer
     wxTimer m_timerIdleWakeUp;
 
 
     // the idle event wake up timer
     wxTimer m_timerIdleWakeUp;
 
-    // any class wishing to process wxWindows events must use this macro
+    // a background timer allowing to easily check visually whether the
+    // messages are processed or not
+    wxTimer m_timerBg;
+
+    // any class wishing to process wxWidgets events must use this macro
     DECLARE_EVENT_TABLE()
 };
 
     DECLARE_EVENT_TABLE()
 };
 
@@ -194,7 +207,9 @@ public:
 protected:
     void OnTextEnter(wxCommandEvent& WXUNUSED(event)) { DoSend(); }
     void OnBtnSend(wxCommandEvent& WXUNUSED(event)) { DoSend(); }
 protected:
     void OnTextEnter(wxCommandEvent& WXUNUSED(event)) { DoSend(); }
     void OnBtnSend(wxCommandEvent& WXUNUSED(event)) { DoSend(); }
+    void OnBtnSendFile(wxCommandEvent& WXUNUSED(event));
     void OnBtnGet(wxCommandEvent& WXUNUSED(event)) { DoGet(); }
     void OnBtnGet(wxCommandEvent& WXUNUSED(event)) { DoGet(); }
+    void OnBtnClose(wxCommandEvent& WXUNUSED(event)) { DoClose(); }
 
     void OnClose(wxCloseEvent& event);
 
 
     void OnClose(wxCloseEvent& event);
 
@@ -202,22 +217,32 @@ protected:
 
     void DoSend()
     {
 
     void DoSend()
     {
-        m_out.WriteString(m_textIn->GetValue() + _T('\n'));
-        m_textIn->Clear();
+        wxString s(m_textOut->GetValue());
+        s += wxT('\n');
+        m_out.Write(s.c_str(), s.length());
+        m_textOut->Clear();
 
         DoGet();
     }
 
     void DoGet();
 
         DoGet();
     }
 
     void DoGet();
+    void DoClose();
 
 private:
 
 private:
+    void DoGetFromStream(wxTextCtrl *text, wxInputStream& in);
+    void DisableInput();
+    void DisableOutput();
+
+
     wxProcess *m_process;
 
     wxProcess *m_process;
 
-    wxTextInputStream m_in;
-    wxTextOutputStream m_out;
+    wxOutputStream &m_out;
+    wxInputStream &m_in,
+                  &m_err;
 
 
-    wxTextCtrl *m_textIn,
-               *m_textOut;
+    wxTextCtrl *m_textOut,
+               *m_textIn,
+               *m_textErr;
 
     DECLARE_EVENT_TABLE()
 };
 
     DECLARE_EVENT_TABLE()
 };
@@ -284,39 +309,55 @@ private:
 // IDs for the controls and the menu commands
 enum
 {
 // IDs for the controls and the menu commands
 enum
 {
+    // timer ids
+    Exec_TimerIdle = 10,
+    Exec_TimerBg,
+
     // menu items
     // menu items
-    Exec_Quit = 100,
-    Exec_Kill,
+    Exec_Kill = 100,
     Exec_ClearLog,
     Exec_ClearLog,
+    Exec_BeginBusyCursor,
+    Exec_EndBusyCursor,
     Exec_SyncExec = 200,
     Exec_AsyncExec,
     Exec_Shell,
     Exec_POpen,
     Exec_OpenFile,
     Exec_SyncExec = 200,
     Exec_AsyncExec,
     Exec_Shell,
     Exec_POpen,
     Exec_OpenFile,
+    Exec_ShowCommandForExt,
+    Exec_LaunchFile,
+    Exec_OpenURL,
     Exec_DDEExec,
     Exec_DDERequest,
     Exec_Redirect,
     Exec_Pipe,
     Exec_DDEExec,
     Exec_DDERequest,
     Exec_Redirect,
     Exec_Pipe,
-    Exec_About = 300,
+    Exec_Flags_HideConsole,
+    Exec_Flags_ShowConsole,
+    Exec_Flags_NoEvents,
+    Exec_About = wxID_ABOUT,
+    Exec_Quit = wxID_EXIT,
 
     // control ids
     Exec_Btn_Send = 1000,
 
     // control ids
     Exec_Btn_Send = 1000,
-    Exec_Btn_Get
+    Exec_Btn_SendFile,
+    Exec_Btn_Get,
+    Exec_Btn_Close
 };
 
 };
 
-static const wxChar *DIALOG_TITLE = _T("Exec sample");
+static const wxChar *DIALOG_TITLE = wxT("Exec sample");
 
 // ----------------------------------------------------------------------------
 
 // ----------------------------------------------------------------------------
-// event tables and other macros for wxWindows
+// event tables and other macros for wxWidgets
 // ----------------------------------------------------------------------------
 
 // ----------------------------------------------------------------------------
 
-// the event tables connect the wxWindows events with the functions (event
+// the event tables connect the wxWidgets events with the functions (event
 // handlers) which process them. It can be also done at run-time, but for the
 // simple menu events like this the static method is much simpler.
 BEGIN_EVENT_TABLE(MyFrame, wxFrame)
     EVT_MENU(Exec_Quit,  MyFrame::OnQuit)
     EVT_MENU(Exec_Kill,  MyFrame::OnKill)
     EVT_MENU(Exec_ClearLog,  MyFrame::OnClear)
 // handlers) which process them. It can be also done at run-time, but for the
 // simple menu events like this the static method is much simpler.
 BEGIN_EVENT_TABLE(MyFrame, wxFrame)
     EVT_MENU(Exec_Quit,  MyFrame::OnQuit)
     EVT_MENU(Exec_Kill,  MyFrame::OnKill)
     EVT_MENU(Exec_ClearLog,  MyFrame::OnClear)
+    EVT_MENU(Exec_BeginBusyCursor,  MyFrame::OnBeginBusyCursor)
+    EVT_MENU(Exec_EndBusyCursor,  MyFrame::OnEndBusyCursor)
 
     EVT_MENU(Exec_SyncExec, MyFrame::OnSyncExec)
     EVT_MENU(Exec_AsyncExec, MyFrame::OnAsyncExec)
 
     EVT_MENU(Exec_SyncExec, MyFrame::OnSyncExec)
     EVT_MENU(Exec_AsyncExec, MyFrame::OnAsyncExec)
@@ -327,6 +368,9 @@ BEGIN_EVENT_TABLE(MyFrame, wxFrame)
     EVT_MENU(Exec_POpen, MyFrame::OnPOpen)
 
     EVT_MENU(Exec_OpenFile, MyFrame::OnFileExec)
     EVT_MENU(Exec_POpen, MyFrame::OnPOpen)
 
     EVT_MENU(Exec_OpenFile, MyFrame::OnFileExec)
+    EVT_MENU(Exec_ShowCommandForExt, MyFrame::OnShowCommandForExt)
+    EVT_MENU(Exec_LaunchFile, MyFrame::OnFileLaunch)
+    EVT_MENU(Exec_OpenURL, MyFrame::OnOpenURL)
 
 #ifdef __WINDOWS__
     EVT_MENU(Exec_DDEExec, MyFrame::OnDDEExec)
 
 #ifdef __WINDOWS__
     EVT_MENU(Exec_DDEExec, MyFrame::OnDDEExec)
@@ -337,21 +381,24 @@ BEGIN_EVENT_TABLE(MyFrame, wxFrame)
 
     EVT_IDLE(MyFrame::OnIdle)
 
 
     EVT_IDLE(MyFrame::OnIdle)
 
-    EVT_TIMER(-1, MyFrame::OnTimer)
+    EVT_TIMER(Exec_TimerIdle, MyFrame::OnIdleTimer)
+    EVT_TIMER(Exec_TimerBg, MyFrame::OnBgTimer)
 END_EVENT_TABLE()
 
 BEGIN_EVENT_TABLE(MyPipeFrame, wxFrame)
     EVT_BUTTON(Exec_Btn_Send, MyPipeFrame::OnBtnSend)
 END_EVENT_TABLE()
 
 BEGIN_EVENT_TABLE(MyPipeFrame, wxFrame)
     EVT_BUTTON(Exec_Btn_Send, MyPipeFrame::OnBtnSend)
+    EVT_BUTTON(Exec_Btn_SendFile, MyPipeFrame::OnBtnSendFile)
     EVT_BUTTON(Exec_Btn_Get, MyPipeFrame::OnBtnGet)
     EVT_BUTTON(Exec_Btn_Get, MyPipeFrame::OnBtnGet)
+    EVT_BUTTON(Exec_Btn_Close, MyPipeFrame::OnBtnClose)
 
 
-    EVT_TEXT_ENTER(-1, MyPipeFrame::OnTextEnter)
+    EVT_TEXT_ENTER(wxID_ANY, MyPipeFrame::OnTextEnter)
 
     EVT_CLOSE(MyPipeFrame::OnClose)
 
 
     EVT_CLOSE(MyPipeFrame::OnClose)
 
-    EVT_END_PROCESS(-1, MyPipeFrame::OnProcessTerm)
+    EVT_END_PROCESS(wxID_ANY, MyPipeFrame::OnProcessTerm)
 END_EVENT_TABLE()
 
 END_EVENT_TABLE()
 
-// Create a new application object: this macro will allow wxWindows to create
+// Create a new application object: this macro will allow wxWidgets to create
 // the application object during program execution (it's better than using a
 // static object for many reasons) and also declares the accessor function
 // wxGetApp() which will return the reference of the right type (i.e. MyApp and
 // the application object during program execution (it's better than using a
 // static object for many reasons) and also declares the accessor function
 // wxGetApp() which will return the reference of the right type (i.e. MyApp and
@@ -369,18 +416,20 @@ IMPLEMENT_APP(MyApp)
 // `Main program' equivalent: the program execution "starts" here
 bool MyApp::OnInit()
 {
 // `Main program' equivalent: the program execution "starts" here
 bool MyApp::OnInit()
 {
+    if ( !wxApp::OnInit() )
+        return false;
+
     // Create the main application window
     // Create the main application window
-    MyFrame *frame = new MyFrame(_T("Exec wxWindows sample"),
+    MyFrame *frame = new MyFrame(wxT("Exec wxWidgets sample"),
                                  wxDefaultPosition, wxSize(500, 140));
 
                                  wxDefaultPosition, wxSize(500, 140));
 
-    // Show it and tell the application that it's our main window
-    frame->Show(TRUE);
-    SetTopWindow(frame);
+    // Show it
+    frame->Show(true);
 
     // success: wxApp::OnRun() will be called which will enter the main message
 
     // success: wxApp::OnRun() will be called which will enter the main message
-    // loop and the application will run. If we returned FALSE here, the
+    // loop and the application will run. If we returned false here, the
     // application would exit immediately.
     // application would exit immediately.
-    return TRUE;
+    return true;
 }
 
 // ----------------------------------------------------------------------------
 }
 
 // ----------------------------------------------------------------------------
@@ -393,9 +442,12 @@ bool MyApp::OnInit()
 
 // frame constructor
 MyFrame::MyFrame(const wxString& title, const wxPoint& pos, const wxSize& size)
 
 // frame constructor
 MyFrame::MyFrame(const wxString& title, const wxPoint& pos, const wxSize& size)
-       : wxFrame((wxFrame *)NULL, -1, title, pos, size),
-         m_timerIdleWakeUp(this)
+       : wxFrame((wxFrame *)NULL, wxID_ANY, title, pos, size),
+         m_timerIdleWakeUp(this, Exec_TimerIdle),
+         m_timerBg(this, Exec_TimerBg)
 {
 {
+    SetIcon(wxICON(sample));
+
     m_pidLast = 0;
 
 #ifdef __WXMAC__
     m_pidLast = 0;
 
 #ifdef __WXMAC__
@@ -405,63 +457,94 @@ MyFrame::MyFrame(const wxString& title, const wxPoint& pos, const wxSize& size)
 #endif
 
     // create a menu bar
 #endif
 
     // create a menu bar
-    wxMenu *menuFile = new wxMenu(_T(""), wxMENU_TEAROFF);
-    menuFile->Append(Exec_Kill, _T("&Kill process...\tCtrl-K"),
-                     _T("Kill a process by PID"));
+    wxMenu *menuFile = new wxMenu(wxEmptyString, wxMENU_TEAROFF);
+    menuFile->Append(Exec_Kill, wxT("&Kill process...\tCtrl-K"),
+                     wxT("Kill a process by PID"));
     menuFile->AppendSeparator();
     menuFile->AppendSeparator();
-    menuFile->Append(Exec_ClearLog, _T("&Clear log\tCtrl-C"),
-                     _T("Clear the log window"));
+    menuFile->Append(Exec_OpenFile, wxT("Open &file...\tCtrl-F"),
+                     wxT("Launch the command to open this kind of files"));
+    menuFile->Append(Exec_ShowCommandForExt,
+                     "Show association for extension...\tShift-Ctrl-A",
+                     "Show the command associated with the given extension");
+    menuFile->Append(Exec_LaunchFile, wxT("La&unch file...\tShift-Ctrl-F"),
+                     wxT("Launch the default application associated with the file"));
+    menuFile->Append(Exec_OpenURL, wxT("Open &URL...\tCtrl-U"),
+                     wxT("Launch the default browser with the given URL"));
     menuFile->AppendSeparator();
     menuFile->AppendSeparator();
-    menuFile->Append(Exec_Quit, _T("E&xit\tAlt-X"), _T("Quit this program"));
+    menuFile->Append(Exec_BeginBusyCursor, wxT("Show &busy cursor\tCtrl-C"));
+    menuFile->Append(Exec_EndBusyCursor, wxT("Show &normal cursor\tShift-Ctrl-C"));
+    menuFile->AppendSeparator();
+    menuFile->Append(Exec_ClearLog, wxT("&Clear log\tCtrl-L"),
+                     wxT("Clear the log window"));
+    menuFile->AppendSeparator();
+    menuFile->Append(Exec_Quit, wxT("E&xit\tAlt-X"), wxT("Quit this program"));
+
+    wxMenu *flagsMenu = new wxMenu;
+    flagsMenu->AppendCheckItem(Exec_Flags_HideConsole, "Always &hide console");
+    flagsMenu->AppendCheckItem(Exec_Flags_ShowConsole, "Always &show console");
+    flagsMenu->AppendCheckItem(Exec_Flags_NoEvents, "Disable &events",
+                               "This flag is valid for sync execution only");
 
     wxMenu *execMenu = new wxMenu;
 
     wxMenu *execMenu = new wxMenu;
-    execMenu->Append(Exec_SyncExec, _T("Sync &execution...\tCtrl-E"),
-                     _T("Launch a program and return when it terminates"));
-    execMenu->Append(Exec_AsyncExec, _T("&Async execution...\tCtrl-A"),
-                     _T("Launch a program and return immediately"));
-    execMenu->Append(Exec_Shell, _T("Execute &shell command...\tCtrl-S"),
-                     _T("Launch a shell and execute a command in it"));
+    execMenu->AppendSubMenu(flagsMenu, "Execution flags");
     execMenu->AppendSeparator();
     execMenu->AppendSeparator();
-    execMenu->Append(Exec_Redirect, _T("Capture command &output...\tCtrl-O"),
-                     _T("Launch a program and capture its output"));
-    execMenu->Append(Exec_Pipe, _T("&Pipe through command..."),
-                     _T("Pipe a string through a filter"));
-    execMenu->Append(Exec_POpen, _T("&Open a pipe to a command...\tCtrl-P"),
-                     _T("Open a pipe to and from another program"));
-
+    execMenu->Append(Exec_SyncExec, wxT("Sync &execution...\tCtrl-E"),
+                     wxT("Launch a program and return when it terminates"));
+    execMenu->Append(Exec_AsyncExec, wxT("&Async execution...\tCtrl-A"),
+                     wxT("Launch a program and return immediately"));
+    execMenu->Append(Exec_Shell, wxT("Execute &shell command...\tCtrl-S"),
+                     wxT("Launch a shell and execute a command in it"));
     execMenu->AppendSeparator();
     execMenu->AppendSeparator();
-    execMenu->Append(Exec_OpenFile, _T("Open &file...\tCtrl-F"),
-                     _T("Launch the command to open this kind of files"));
+    execMenu->Append(Exec_Redirect, wxT("Capture command &output...\tCtrl-O"),
+                     wxT("Launch a program and capture its output"));
+    execMenu->Append(Exec_Pipe, wxT("&Pipe through command..."),
+                     wxT("Pipe a string through a filter"));
+    execMenu->Append(Exec_POpen, wxT("&Open a pipe to a command...\tCtrl-P"),
+                     wxT("Open a pipe to and from another program"));
+
 #ifdef __WINDOWS__
     execMenu->AppendSeparator();
 #ifdef __WINDOWS__
     execMenu->AppendSeparator();
-    execMenu->Append(Exec_DDEExec, _T("Execute command via &DDE...\tCtrl-D"));
-    execMenu->Append(Exec_DDERequest, _T("Send DDE &request...\tCtrl-R"));
+    execMenu->Append(Exec_DDEExec, wxT("Execute command via &DDE...\tCtrl-D"));
+    execMenu->Append(Exec_DDERequest, wxT("Send DDE &request...\tCtrl-R"));
 #endif
 
 #endif
 
-    wxMenu *helpMenu = new wxMenu(_T(""), wxMENU_TEAROFF);
-    helpMenu->Append(Exec_About, _T("&About...\tF1"), _T("Show about dialog"));
+    wxMenu *helpMenu = new wxMenu(wxEmptyString, wxMENU_TEAROFF);
+    helpMenu->Append(Exec_About, wxT("&About\tF1"), wxT("Show about dialog"));
 
     // now append the freshly created menu to the menu bar...
     wxMenuBar *menuBar = new wxMenuBar();
 
     // now append the freshly created menu to the menu bar...
     wxMenuBar *menuBar = new wxMenuBar();
-    menuBar->Append(menuFile, _T("&File"));
-    menuBar->Append(execMenu, _T("&Exec"));
-    menuBar->Append(helpMenu, _T("&Help"));
+    menuBar->Append(menuFile, wxT("&File"));
+    menuBar->Append(execMenu, wxT("&Exec"));
+    menuBar->Append(helpMenu, wxT("&Help"));
 
     // ... and attach this menu bar to the frame
     SetMenuBar(menuBar);
 
     // create the listbox in which we will show misc messages as they come
 
     // ... and attach this menu bar to the frame
     SetMenuBar(menuBar);
 
     // create the listbox in which we will show misc messages as they come
-    m_lbox = new wxListBox(this, -1);
+    m_lbox = new wxListBox(this, wxID_ANY);
     wxFont font(12, wxFONTFAMILY_TELETYPE, wxFONTSTYLE_NORMAL,
                 wxFONTWEIGHT_NORMAL);
     wxFont font(12, wxFONTFAMILY_TELETYPE, wxFONTSTYLE_NORMAL,
                 wxFONTWEIGHT_NORMAL);
-    if ( font.Ok() )
+    if ( font.IsOk() )
         m_lbox->SetFont(font);
 
 #if wxUSE_STATUSBAR
     // create a status bar just for fun (by default with 1 pane only)
         m_lbox->SetFont(font);
 
 #if wxUSE_STATUSBAR
     // create a status bar just for fun (by default with 1 pane only)
-    CreateStatusBar();
-    SetStatusText(_T("Welcome to wxWindows exec sample!"));
+    CreateStatusBar(2);
+    SetStatusText(wxT("Welcome to wxWidgets exec sample!"));
 #endif // wxUSE_STATUSBAR
 #endif // wxUSE_STATUSBAR
+
+    m_timerBg.Start(1000);
+}
+
+MyFrame::~MyFrame()
+{
+    // any processes left until now must be deleted manually: normally this is
+    // done when the associated process terminates but it must be still running
+    // if this didn't happen until now
+    for ( size_t n = 0; n < m_allAsync.size(); n++ )
+    {
+        delete m_allAsync[n];
+    }
 }
 
 // ----------------------------------------------------------------------------
 }
 
 // ----------------------------------------------------------------------------
@@ -470,8 +553,8 @@ MyFrame::MyFrame(const wxString& title, const wxPoint& pos, const wxSize& size)
 
 void MyFrame::OnQuit(wxCommandEvent& WXUNUSED(event))
 {
 
 void MyFrame::OnQuit(wxCommandEvent& WXUNUSED(event))
 {
-    // TRUE is to force the frame to close
-    Close(TRUE);
+    // true is to force the frame to close
+    Close(true);
 }
 
 void MyFrame::OnClear(wxCommandEvent& WXUNUSED(event))
 }
 
 void MyFrame::OnClear(wxCommandEvent& WXUNUSED(event))
@@ -479,17 +562,27 @@ void MyFrame::OnClear(wxCommandEvent& WXUNUSED(event))
     m_lbox->Clear();
 }
 
     m_lbox->Clear();
 }
 
+void MyFrame::OnBeginBusyCursor(wxCommandEvent& WXUNUSED(event))
+{
+    wxBeginBusyCursor();
+}
+
+void MyFrame::OnEndBusyCursor(wxCommandEvent& WXUNUSED(event))
+{
+    wxEndBusyCursor();
+}
+
 void MyFrame::OnAbout(wxCommandEvent& WXUNUSED(event))
 {
 void MyFrame::OnAbout(wxCommandEvent& WXUNUSED(event))
 {
-    wxMessageBox(_T("Exec wxWindows Sample\n© 2000-2002 Vadim Zeitlin"),
-                 _T("About Exec"), wxOK | wxICON_INFORMATION, this);
+    wxMessageBox(wxT("Exec wxWidgets Sample\n(c) 2000-2002 Vadim Zeitlin"),
+                 wxT("About Exec"), wxOK | wxICON_INFORMATION, this);
 }
 
 void MyFrame::OnKill(wxCommandEvent& WXUNUSED(event))
 {
 }
 
 void MyFrame::OnKill(wxCommandEvent& WXUNUSED(event))
 {
-    long pid = wxGetNumberFromUser(_T("Please specify the process to kill"),
-                                   _T("Enter PID:"),
-                                   _T("Exec question"),
+    long pid = wxGetNumberFromUser(wxT("Please specify the process to kill"),
+                                   wxT("Enter PID:"),
+                                   wxT("Exec question"),
                                    m_pidLast,
                                    // we need the full unsigned int range
                                    -INT_MAX, INT_MAX,
                                    m_pidLast,
                                    // we need the full unsigned int range
                                    -INT_MAX, INT_MAX,
@@ -500,34 +593,38 @@ void MyFrame::OnKill(wxCommandEvent& WXUNUSED(event))
         return;
     }
 
         return;
     }
 
+    m_pidLast = pid;
+
     static const wxString signalNames[] =
     {
     static const wxString signalNames[] =
     {
-        _T("Just test (SIGNONE)"),
-        _T("Hangup (SIGHUP)"),
-        _T("Interrupt (SIGINT)"),
-        _T("Quit (SIGQUIT)"),
-        _T("Illegal instruction (SIGILL)"),
-        _T("Trap (SIGTRAP)"),
-        _T("Abort (SIGABRT)"),
-        _T("Emulated trap (SIGEMT)"),
-        _T("FP exception (SIGFPE)"),
-        _T("Kill (SIGKILL)"),
-        _T("Bus (SIGBUS)"),
-        _T("Segment violation (SIGSEGV)"),
-        _T("System (SIGSYS)"),
-        _T("Broken pipe (SIGPIPE)"),
-        _T("Alarm (SIGALRM)"),
-        _T("Terminate (SIGTERM)"),
+        wxT("Just test (SIGNONE)"),
+        wxT("Hangup (SIGHUP)"),
+        wxT("Interrupt (SIGINT)"),
+        wxT("Quit (SIGQUIT)"),
+        wxT("Illegal instruction (SIGILL)"),
+        wxT("Trap (SIGTRAP)"),
+        wxT("Abort (SIGABRT)"),
+        wxT("Emulated trap (SIGEMT)"),
+        wxT("FP exception (SIGFPE)"),
+        wxT("Kill (SIGKILL)"),
+        wxT("Bus (SIGBUS)"),
+        wxT("Segment violation (SIGSEGV)"),
+        wxT("System (SIGSYS)"),
+        wxT("Broken pipe (SIGPIPE)"),
+        wxT("Alarm (SIGALRM)"),
+        wxT("Terminate (SIGTERM)"),
     };
 
     };
 
-    int sig = wxGetSingleChoiceIndex(_T("How to kill the process?"),
-                                     _T("Exec question"),
+    static int s_sigLast = wxSIGNONE;
+    int sig = wxGetSingleChoiceIndex(wxT("How to kill the process?"),
+                                     wxT("Exec question"),
                                      WXSIZEOF(signalNames), signalNames,
                                      WXSIZEOF(signalNames), signalNames,
+                                     s_sigLast,
                                      this);
     switch ( sig )
     {
         default:
                                      this);
     switch ( sig )
     {
         default:
-            wxFAIL_MSG( _T("unexpected return value") );
+            wxFAIL_MSG( wxT("unexpected return value") );
             // fall through
 
         case -1:
             // fall through
 
         case -1:
@@ -553,74 +650,239 @@ void MyFrame::OnKill(wxCommandEvent& WXUNUSED(event))
             break;
     }
 
             break;
     }
 
-    if ( sig == 0 )
+    s_sigLast = sig;
+
+    if ( sig == wxSIGNONE )
     {
     {
+        // This simply calls Kill(wxSIGNONE) but using it is more convenient.
         if ( wxProcess::Exists(pid) )
         if ( wxProcess::Exists(pid) )
-            wxLogStatus(_T("Process %ld is running."), pid);
+        {
+            wxLogStatus(wxT("Process %ld is running."), pid);
+        }
         else
         else
-            wxLogStatus(_T("No process with pid = %ld."), pid);
+        {
+            wxLogStatus(wxT("No process with pid = %ld."), pid);
+        }
     }
     else // not SIGNONE
     {
         wxKillError rc = wxProcess::Kill(pid, (wxSignal)sig);
         if ( rc == wxKILL_OK )
         {
     }
     else // not SIGNONE
     {
         wxKillError rc = wxProcess::Kill(pid, (wxSignal)sig);
         if ( rc == wxKILL_OK )
         {
-            wxLogStatus(_T("Process %ld killed with signal %d."), pid, sig);
+            wxLogStatus(wxT("Process %ld killed with signal %d."), pid, sig);
         }
         else
         {
             static const wxChar *errorText[] =
             {
         }
         else
         {
             static const wxChar *errorText[] =
             {
-                _T(""), // no error
-                _T("signal not supported"),
-                _T("permission denied"),
-                _T("no such process"),
-                _T("unspecified error"),
+                wxT(""), // no error
+                wxT("signal not supported"),
+                wxT("permission denied"),
+                wxT("no such process"),
+                wxT("unspecified error"),
             };
 
             };
 
-            wxLogStatus(_T("Failed to kill process %ld with signal %d: %s"),
+            wxLogStatus(wxT("Failed to kill process %ld with signal %d: %s"),
                         pid, sig, errorText[rc]);
         }
     }
 }
 
                         pid, sig, errorText[rc]);
         }
     }
 }
 
+// ----------------------------------------------------------------------------
+// execution options dialog
+// ----------------------------------------------------------------------------
+
+enum ExecQueryDialogID
+{
+    TEXT_EXECUTABLE,
+    TEXT_CWD,
+    TEXT_ENVIRONMENT
+};
+
+class ExecQueryDialog : public wxDialog
+{
+public:
+    ExecQueryDialog(const wxString& cmd);
+
+    wxString GetExecutable() const
+    {
+        return m_executable->GetValue();
+    }
+
+    wxString GetWorkDir() const
+    {
+        return m_useCWD->GetValue() ? m_cwdtext->GetValue() : wxString();
+    }
+
+    void GetEnvironment(wxEnvVariableHashMap& env);
+
+private:
+    void OnUpdateWorkingDirectoryUI(wxUpdateUIEvent& event)
+    {
+        event.Enable(m_useCWD->GetValue());
+    }
+
+    void OnUpdateEnvironmentUI(wxUpdateUIEvent& event)
+    {
+        event.Enable(m_useEnv->GetValue());
+    }
+
+    wxTextCtrl* m_executable;
+    wxTextCtrl* m_cwdtext;
+    wxTextCtrl* m_envtext;
+    wxCheckBox* m_useCWD;
+    wxCheckBox* m_useEnv;
+
+    DECLARE_EVENT_TABLE()
+};
+
+BEGIN_EVENT_TABLE(ExecQueryDialog, wxDialog)
+    EVT_UPDATE_UI(TEXT_CWD, ExecQueryDialog::OnUpdateWorkingDirectoryUI)
+    EVT_UPDATE_UI(TEXT_ENVIRONMENT, ExecQueryDialog::OnUpdateEnvironmentUI)
+END_EVENT_TABLE()
+
+ExecQueryDialog::ExecQueryDialog(const wxString& cmd)
+    : wxDialog(NULL, wxID_ANY, DIALOG_TITLE,
+               wxDefaultPosition, wxDefaultSize,
+               wxDEFAULT_DIALOG_STYLE | wxRESIZE_BORDER)
+{
+    wxSizer* globalSizer = new wxBoxSizer(wxVERTICAL);
+
+    m_executable = new wxTextCtrl(this, TEXT_EXECUTABLE, wxString());
+    m_cwdtext = new wxTextCtrl(this, TEXT_CWD, wxString());
+    m_envtext = new wxTextCtrl(this, TEXT_ENVIRONMENT, wxString(),
+                               wxDefaultPosition, wxSize(300, 200),
+                               wxTE_MULTILINE|wxHSCROLL);
+
+    const wxSizerFlags flagsExpand = wxSizerFlags().Expand().Border();
+    globalSizer->Add(new wxStaticText(this, wxID_ANY, "Enter the command: "),
+                     flagsExpand);
+    globalSizer->Add(m_executable, flagsExpand);
+
+    m_useCWD = new wxCheckBox(this, wxID_ANY, "Working directory: ");
+    globalSizer->Add(m_useCWD, flagsExpand);
+    globalSizer->Add(m_cwdtext, flagsExpand);
+
+    m_useEnv = new wxCheckBox(this, wxID_ANY, "Environment: ");
+    globalSizer->Add(m_useEnv, flagsExpand);
+    globalSizer->Add(m_envtext, wxSizerFlags(flagsExpand).Proportion(1));
+
+    globalSizer->Add(CreateStdDialogButtonSizer(wxOK|wxCANCEL), flagsExpand);
+    SetSizerAndFit(globalSizer);
+
+
+    m_executable->SetValue(cmd);
+    m_cwdtext->SetValue(wxGetCwd());
+    wxEnvVariableHashMap env;
+    if ( wxGetEnvMap(&env) )
+    {
+        for ( wxEnvVariableHashMap::iterator it = env.begin();
+              it != env.end();
+              ++it )
+        {
+            m_envtext->AppendText(it->first + '=' + it->second + '\n');
+        }
+    }
+    m_useCWD->SetValue(false);
+    m_useEnv->SetValue(false);
+}
+
+void ExecQueryDialog::GetEnvironment(wxEnvVariableHashMap& env)
+{
+    env.clear();
+    if ( m_useEnv->GetValue() )
+    {
+        wxString name,
+                 value;
+
+        const int nb = m_envtext->GetNumberOfLines();
+        for ( int l = 0; l < nb; l++ )
+        {
+            const wxString line = m_envtext->GetLineText(l).Trim();
+
+            if ( !line.empty() )
+            {
+                name = line.BeforeFirst('=', &value);
+                if ( name.empty() )
+                {
+                    wxLogWarning("Skipping invalid environment line \"%s\".", line);
+                    continue;
+                }
+
+                env[name] = value;
+            }
+        }
+    }
+}
+
+static bool QueryExec(wxString& cmd, wxExecuteEnv& env)
+{
+    ExecQueryDialog dialog(cmd);
+
+    if ( dialog.ShowModal() != wxID_OK )
+        return false;
+
+    cmd = dialog.GetExecutable();
+    env.cwd = dialog.GetWorkDir();
+    dialog.GetEnvironment(env.env);
+
+    return true;
+}
+
 // ----------------------------------------------------------------------------
 // event handlers: exec menu
 // ----------------------------------------------------------------------------
 
 // ----------------------------------------------------------------------------
 // event handlers: exec menu
 // ----------------------------------------------------------------------------
 
+int MyFrame::GetExecFlags() const
+{
+    wxMenuBar* const mbar = GetMenuBar();
+
+    int flags = 0;
+
+    if ( mbar->IsChecked(Exec_Flags_HideConsole) )
+        flags |= wxEXEC_HIDE_CONSOLE;
+    if ( mbar->IsChecked(Exec_Flags_ShowConsole) )
+        flags |= wxEXEC_SHOW_CONSOLE;
+    if ( mbar->IsChecked(Exec_Flags_NoEvents) )
+        flags |= wxEXEC_NOEVENTS;
+
+    return flags;
+}
+
 void MyFrame::DoAsyncExec(const wxString& cmd)
 {
 void MyFrame::DoAsyncExec(const wxString& cmd)
 {
-    wxProcess *process = new MyProcess(this, cmd);
-    m_pidLast = wxExecute(cmd, wxEXEC_ASYNC, process);
+    MyProcess * const process = new MyProcess(this, cmd);
+    m_pidLast = wxExecute(cmd, wxEXEC_ASYNC | GetExecFlags(), process);
     if ( !m_pidLast )
     {
     if ( !m_pidLast )
     {
-        wxLogError( _T("Execution of '%s' failed."), cmd.c_str() );
+        wxLogError(wxT("Execution of '%s' failed."), cmd.c_str());
 
         delete process;
     }
     else
     {
 
         delete process;
     }
     else
     {
-        wxLogStatus( _T("Process %ld (%s) launched."),
-            m_pidLast, cmd.c_str() );
+        wxLogStatus(wxT("Process %ld (%s) launched."), m_pidLast, cmd.c_str());
 
         m_cmdLast = cmd;
 
         m_cmdLast = cmd;
+
+        // the parent frame keeps track of all async processes as it needs to
+        // free them if we exit before the child process terminates
+        AddAsyncProcess(process);
     }
 }
 
 void MyFrame::OnSyncExec(wxCommandEvent& WXUNUSED(event))
 {
     }
 }
 
 void MyFrame::OnSyncExec(wxCommandEvent& WXUNUSED(event))
 {
-    wxString cmd = wxGetTextFromUser(_T("Enter the command: "),
-                                     DIALOG_TITLE,
-                                     m_cmdLast);
-
-    if ( !cmd )
+    wxString cmd;
+    wxExecuteEnv env;
+    if ( !QueryExec(cmd, env) )
         return;
 
         return;
 
-    wxLogStatus( _T("'%s' is running please wait..."), cmd.c_str() );
+    wxLogStatus( wxT("'%s' is running please wait..."), cmd.c_str() );
 
 
-    int code = wxExecute(cmd, wxEXEC_SYNC);
+    int code = wxExecute(cmd, wxEXEC_SYNC | GetExecFlags(), NULL, &env);
 
 
-    wxLogStatus(_T("Process '%s' terminated with exit code %d."),
+    wxLogStatus(wxT("Process '%s' terminated with exit code %d."),
         cmd.c_str(), code);
 
     m_cmdLast = cmd;
         cmd.c_str(), code);
 
     m_cmdLast = cmd;
@@ -628,7 +890,7 @@ void MyFrame::OnSyncExec(wxCommandEvent& WXUNUSED(event))
 
 void MyFrame::OnAsyncExec(wxCommandEvent& WXUNUSED(event))
 {
 
 void MyFrame::OnAsyncExec(wxCommandEvent& WXUNUSED(event))
 {
-    wxString cmd = wxGetTextFromUser(_T("Enter the command: "),
+    wxString cmd = wxGetTextFromUser(wxT("Enter the command: "),
                                      DIALOG_TITLE,
                                      m_cmdLast);
 
                                      DIALOG_TITLE,
                                      m_cmdLast);
 
@@ -640,7 +902,7 @@ void MyFrame::OnAsyncExec(wxCommandEvent& WXUNUSED(event))
 
 void MyFrame::OnShell(wxCommandEvent& WXUNUSED(event))
 {
 
 void MyFrame::OnShell(wxCommandEvent& WXUNUSED(event))
 {
-    wxString cmd = wxGetTextFromUser(_T("Enter the command: "),
+    wxString cmd = wxGetTextFromUser(wxT("Enter the command: "),
                                      DIALOG_TITLE,
                                      m_cmdLast);
 
                                      DIALOG_TITLE,
                                      m_cmdLast);
 
@@ -648,14 +910,23 @@ void MyFrame::OnShell(wxCommandEvent& WXUNUSED(event))
         return;
 
     int code = wxShell(cmd);
         return;
 
     int code = wxShell(cmd);
-    wxLogStatus(_T("Shell command '%s' terminated with exit code %d."),
+    wxLogStatus(wxT("Shell command '%s' terminated with exit code %d."),
                 cmd.c_str(), code);
     m_cmdLast = cmd;
 }
 
 void MyFrame::OnExecWithRedirect(wxCommandEvent& WXUNUSED(event))
 {
                 cmd.c_str(), code);
     m_cmdLast = cmd;
 }
 
 void MyFrame::OnExecWithRedirect(wxCommandEvent& WXUNUSED(event))
 {
-    wxString cmd = wxGetTextFromUser(_T("Enter the command: "),
+    if ( !m_cmdLast )
+    {
+#ifdef __WXMSW__
+        m_cmdLast = "type Makefile.in";
+#else
+        m_cmdLast = "cat -n Makefile";
+#endif
+    }
+
+    wxString cmd = wxGetTextFromUser(wxT("Enter the command: "),
                                      DIALOG_TITLE,
                                      m_cmdLast);
 
                                      DIALOG_TITLE,
                                      m_cmdLast);
 
@@ -663,16 +934,16 @@ void MyFrame::OnExecWithRedirect(wxCommandEvent& WXUNUSED(event))
         return;
 
     bool sync;
         return;
 
     bool sync;
-    switch ( wxMessageBox(_T("Execute it synchronously?"),
-                          _T("Exec question"),
+    switch ( wxMessageBox(wxT("Execute it synchronously?"),
+                          wxT("Exec question"),
                           wxYES_NO | wxCANCEL | wxICON_QUESTION, this) )
     {
         case wxYES:
                           wxYES_NO | wxCANCEL | wxICON_QUESTION, this) )
     {
         case wxYES:
-            sync = TRUE;
+            sync = true;
             break;
 
         case wxNO:
             break;
 
         case wxNO:
-            sync = FALSE;
+            sync = false;
             break;
 
         default:
             break;
 
         default:
@@ -681,15 +952,20 @@ void MyFrame::OnExecWithRedirect(wxCommandEvent& WXUNUSED(event))
 
     if ( sync )
     {
 
     if ( sync )
     {
+        wxLogStatus("\"%s\" is running please wait...", cmd);
+
+        wxStopWatch sw;
+
         wxArrayString output, errors;
         int code = wxExecute(cmd, output, errors);
         wxArrayString output, errors;
         int code = wxExecute(cmd, output, errors);
-        wxLogStatus(_T("command '%s' terminated with exit code %d."),
-                    cmd.c_str(), code);
+
+        wxLogStatus("Command \"%s\" terminated after %ldms; exit code %d.",
+                    cmd, sw.Time(), code);
 
         if ( code != -1 )
         {
 
         if ( code != -1 )
         {
-            ShowOutput(cmd, output, _T("Output"));
-            ShowOutput(cmd, errors, _T("Errors"));
+            ShowOutput(cmd, output, wxT("Output"));
+            ShowOutput(cmd, errors, wxT("Errors"));
         }
     }
     else // async exec
         }
     }
     else // async exec
@@ -697,13 +973,13 @@ void MyFrame::OnExecWithRedirect(wxCommandEvent& WXUNUSED(event))
         MyPipedProcess *process = new MyPipedProcess(this, cmd);
         if ( !wxExecute(cmd, wxEXEC_ASYNC, process) )
         {
         MyPipedProcess *process = new MyPipedProcess(this, cmd);
         if ( !wxExecute(cmd, wxEXEC_ASYNC, process) )
         {
-            wxLogError(_T("Execution of '%s' failed."), cmd.c_str());
+            wxLogError(wxT("Execution of '%s' failed."), cmd.c_str());
 
             delete process;
         }
         else
         {
 
             delete process;
         }
         else
         {
-            AddAsyncProcess(process);
+            AddPipedProcess(process);
         }
     }
 
         }
     }
 
@@ -713,16 +989,16 @@ void MyFrame::OnExecWithRedirect(wxCommandEvent& WXUNUSED(event))
 void MyFrame::OnExecWithPipe(wxCommandEvent& WXUNUSED(event))
 {
     if ( !m_cmdLast )
 void MyFrame::OnExecWithPipe(wxCommandEvent& WXUNUSED(event))
 {
     if ( !m_cmdLast )
-        m_cmdLast = _T("tr [a-z] [A-Z]");
+        m_cmdLast = wxT("tr [a-z] [A-Z]");
 
 
-    wxString cmd = wxGetTextFromUser(_T("Enter the command: "),
+    wxString cmd = wxGetTextFromUser(wxT("Enter the command: "),
                                      DIALOG_TITLE,
                                      m_cmdLast);
 
     if ( !cmd )
         return;
 
                                      DIALOG_TITLE,
                                      m_cmdLast);
 
     if ( !cmd )
         return;
 
-    wxString input = wxGetTextFromUser(_T("Enter the string to send to it: "),
+    wxString input = wxGetTextFromUser(wxT("Enter the string to send to it: "),
                                        DIALOG_TITLE);
     if ( !input )
         return;
                                        DIALOG_TITLE);
     if ( !input )
         return;
@@ -732,13 +1008,13 @@ void MyFrame::OnExecWithPipe(wxCommandEvent& WXUNUSED(event))
     long pid = wxExecute(cmd, wxEXEC_ASYNC, process);
     if ( pid )
     {
     long pid = wxExecute(cmd, wxEXEC_ASYNC, process);
     if ( pid )
     {
-        wxLogStatus( _T("Process %ld (%s) launched."), pid, cmd.c_str() );
+        wxLogStatus(wxT("Process %ld (%s) launched."), pid, cmd.c_str());
 
 
-        AddAsyncProcess(process);
+        AddPipedProcess(process);
     }
     else
     {
     }
     else
     {
-        wxLogError(_T("Execution of '%s' failed."), cmd.c_str());
+        wxLogError(wxT("Execution of '%s' failed."), cmd.c_str());
 
         delete process;
     }
 
         delete process;
     }
@@ -748,7 +1024,7 @@ void MyFrame::OnExecWithPipe(wxCommandEvent& WXUNUSED(event))
 
 void MyFrame::OnPOpen(wxCommandEvent& WXUNUSED(event))
 {
 
 void MyFrame::OnPOpen(wxCommandEvent& WXUNUSED(event))
 {
-    wxString cmd = wxGetTextFromUser(_T("Enter the command to launch: "),
+    wxString cmd = wxGetTextFromUser(wxT("Enter the command to launch: "),
                                      DIALOG_TITLE,
                                      m_cmdLast);
     if ( cmd.empty() )
                                      DIALOG_TITLE,
                                      m_cmdLast);
     if ( cmd.empty() )
@@ -757,53 +1033,71 @@ void MyFrame::OnPOpen(wxCommandEvent& WXUNUSED(event))
     wxProcess *process = wxProcess::Open(cmd);
     if ( !process )
     {
     wxProcess *process = wxProcess::Open(cmd);
     if ( !process )
     {
-        wxLogError(_T("Failed to launch the command."));
+        wxLogError(wxT("Failed to launch the command."));
         return;
     }
 
         return;
     }
 
+    wxLogVerbose(wxT("PID of the new process: %ld"), process->GetPid());
+
     wxOutputStream *out = process->GetOutputStream();
     if ( !out )
     {
     wxOutputStream *out = process->GetOutputStream();
     if ( !out )
     {
-        wxLogError(_T("Failed to connect to child stdin"));
+        wxLogError(wxT("Failed to connect to child stdin"));
         return;
     }
 
     wxInputStream *in = process->GetInputStream();
     if ( !in )
     {
         return;
     }
 
     wxInputStream *in = process->GetInputStream();
     if ( !in )
     {
-        wxLogError(_T("Failed to connect to child stdout"));
+        wxLogError(wxT("Failed to connect to child stdout"));
         return;
     }
 
     new MyPipeFrame(this, cmd, process);
 }
 
         return;
     }
 
     new MyPipeFrame(this, cmd, process);
 }
 
-void MyFrame::OnFileExec(wxCommandEvent& WXUNUSED(event))
+static wxString gs_lastFile;
+
+static bool AskUserForFileName()
 {
 {
-    static wxString s_filename;
+    wxString filename;
 
 
-    wxString filename = wxLoadFileSelector(_T(""), _T(""), s_filename);
-    if ( !filename )
-        return;
+#if wxUSE_FILEDLG
+    filename = wxLoadFileSelector(wxT("any"), wxEmptyString, gs_lastFile);
+#else // !wxUSE_FILEDLG
+    filename = wxGetTextFromUser(wxT("Enter the file name"), wxT("exec sample"),
+                                 gs_lastFile);
+#endif // wxUSE_FILEDLG/!wxUSE_FILEDLG
 
 
-    s_filename = filename;
+    if ( filename.empty() )
+        return false;
+
+    gs_lastFile = filename;
+
+    return true;
+}
+
+void MyFrame::OnFileExec(wxCommandEvent& WXUNUSED(event))
+{
+    if ( !AskUserForFileName() )
+        return;
 
 
-    wxString ext = filename.AfterFirst(_T('.'));
+    wxString ext = gs_lastFile.AfterLast(wxT('.'));
     wxFileType *ft = wxTheMimeTypesManager->GetFileTypeFromExtension(ext);
     if ( !ft )
     {
     wxFileType *ft = wxTheMimeTypesManager->GetFileTypeFromExtension(ext);
     if ( !ft )
     {
-        wxLogError(_T("Impossible to determine the file type for extension '%s'"),
+        wxLogError(wxT("Impossible to determine the file type for extension '%s'"),
                    ext.c_str());
         return;
     }
 
     wxString cmd;
     bool ok = ft->GetOpenCommand(&cmd,
                    ext.c_str());
         return;
     }
 
     wxString cmd;
     bool ok = ft->GetOpenCommand(&cmd,
-                                 wxFileType::MessageParameters(filename, _T("")));
+                                 wxFileType::MessageParameters(gs_lastFile));
     delete ft;
     if ( !ok )
     {
     delete ft;
     if ( !ok )
     {
-        wxLogError(_T("Impossible to find out how to open files of extension '%s'"),
+        wxLogError(wxT("Impossible to find out how to open files of extension '%s'"),
                    ext.c_str());
         return;
     }
                    ext.c_str());
         return;
     }
@@ -811,6 +1105,75 @@ void MyFrame::OnFileExec(wxCommandEvent& WXUNUSED(event))
     DoAsyncExec(cmd);
 }
 
     DoAsyncExec(cmd);
 }
 
+void MyFrame::OnShowCommandForExt(wxCommandEvent& WXUNUSED(event))
+{
+    static wxString s_ext;
+
+    wxString ext = wxGetTextFromUser
+                   (
+                    "Enter the extension without leading dot",
+                    "Exec sample",
+                    s_ext,
+                    this
+                   );
+    if ( ext.empty() )
+        return;
+
+    s_ext = ext;
+
+    wxScopedPtr<wxFileType>
+        ft(wxTheMimeTypesManager->GetFileTypeFromExtension(ext));
+    if ( !ft )
+    {
+        wxLogError("Information for extension \"%s\" not found", ext);
+        return;
+    }
+
+    const wxString cmd = ft->GetOpenCommand("file." + ext);
+    if ( cmd.empty() )
+    {
+        wxLogWarning("Open command for extension \"%s\" not defined.", ext);
+        return;
+    }
+
+    wxLogMessage("Open command for files of extension \"%s\" is\n%s",
+                 ext, cmd);
+}
+
+void MyFrame::OnFileLaunch(wxCommandEvent& WXUNUSED(event))
+{
+    if ( !AskUserForFileName() )
+        return;
+
+    if ( !wxLaunchDefaultApplication(gs_lastFile) )
+    {
+        wxLogError("Opening \"%s\" in default application failed.", gs_lastFile);
+    }
+}
+
+void MyFrame::OnOpenURL(wxCommandEvent& WXUNUSED(event))
+{
+    static wxString s_url(wxT("http://www.wxwidgets.org/"));
+
+    wxString filename = wxGetTextFromUser
+                        (
+                            wxT("Enter the URL"),
+                            wxT("exec sample"),
+                            s_url,
+                            this
+                        );
+
+    if ( filename.empty() )
+        return;
+
+    s_url = filename;
+
+    if ( !wxLaunchDefaultBrowser(s_url) )
+    {
+        wxLogError(wxT("Failed to open URL \"%s\""), s_url.c_str());
+    }
+}
+
 // ----------------------------------------------------------------------------
 // DDE stuff
 // ----------------------------------------------------------------------------
 // ----------------------------------------------------------------------------
 // DDE stuff
 // ----------------------------------------------------------------------------
@@ -819,26 +1182,26 @@ void MyFrame::OnFileExec(wxCommandEvent& WXUNUSED(event))
 
 bool MyFrame::GetDDEServer()
 {
 
 bool MyFrame::GetDDEServer()
 {
-    wxString server = wxGetTextFromUser(_T("Server to connect to:"),
+    wxString server = wxGetTextFromUser(wxT("Server to connect to:"),
                                         DIALOG_TITLE, m_server);
     if ( !server )
                                         DIALOG_TITLE, m_server);
     if ( !server )
-        return FALSE;
+        return false;
 
     m_server = server;
 
 
     m_server = server;
 
-    wxString topic = wxGetTextFromUser(_T("DDE topic:"), DIALOG_TITLE, m_topic);
+    wxString topic = wxGetTextFromUser(wxT("DDE topic:"), DIALOG_TITLE, m_topic);
     if ( !topic )
     if ( !topic )
-        return FALSE;
+        return false;
 
     m_topic = topic;
 
 
     m_topic = topic;
 
-    wxString cmd = wxGetTextFromUser(_T("DDE command:"), DIALOG_TITLE, m_cmdDde);
+    wxString cmd = wxGetTextFromUser(wxT("DDE command:"), DIALOG_TITLE, m_cmdDde);
     if ( !cmd )
     if ( !cmd )
-        return FALSE;
+        return false;
 
     m_cmdDde = cmd;
 
 
     m_cmdDde = cmd;
 
-    return TRUE;
+    return true;
 }
 
 void MyFrame::OnDDEExec(wxCommandEvent& WXUNUSED(event))
 }
 
 void MyFrame::OnDDEExec(wxCommandEvent& WXUNUSED(event))
@@ -847,22 +1210,22 @@ void MyFrame::OnDDEExec(wxCommandEvent& WXUNUSED(event))
         return;
 
     wxDDEClient client;
         return;
 
     wxDDEClient client;
-    wxConnectionBase *conn = client.MakeConnection(_T(""), m_server, m_topic);
+    wxConnectionBase *conn = client.MakeConnection(wxEmptyString, m_server, m_topic);
     if ( !conn )
     {
     if ( !conn )
     {
-        wxLogError(_T("Failed to connect to the DDE server '%s'."),
+        wxLogError(wxT("Failed to connect to the DDE server '%s'."),
                    m_server.c_str());
     }
     else
     {
         if ( !conn->Execute(m_cmdDde) )
         {
                    m_server.c_str());
     }
     else
     {
         if ( !conn->Execute(m_cmdDde) )
         {
-            wxLogError(_T("Failed to execute command '%s' via DDE."),
+            wxLogError(wxT("Failed to execute command '%s' via DDE."),
                        m_cmdDde.c_str());
         }
         else
         {
                        m_cmdDde.c_str());
         }
         else
         {
-            wxLogStatus(_T("Successfully executed DDE command"));
+            wxLogStatus(wxT("Successfully executed DDE command"));
         }
     }
 }
         }
     }
 }
@@ -873,22 +1236,22 @@ void MyFrame::OnDDERequest(wxCommandEvent& WXUNUSED(event))
         return;
 
     wxDDEClient client;
         return;
 
     wxDDEClient client;
-    wxConnectionBase *conn = client.MakeConnection(_T(""), m_server, m_topic);
+    wxConnectionBase *conn = client.MakeConnection(wxEmptyString, m_server, m_topic);
     if ( !conn )
     {
     if ( !conn )
     {
-        wxLogError(_T("Failed to connect to the DDE server '%s'."),
+        wxLogError(wxT("Failed to connect to the DDE server '%s'."),
                    m_server.c_str());
     }
     else
     {
         if ( !conn->Request(m_cmdDde) )
         {
                    m_server.c_str());
     }
     else
     {
         if ( !conn->Request(m_cmdDde) )
         {
-            wxLogError(_T("Failed to  send request '%s' via DDE."),
+            wxLogError(wxT("Failed to  send request '%s' via DDE."),
                        m_cmdDde.c_str());
         }
         else
         {
                        m_cmdDde.c_str());
         }
         else
         {
-            wxLogStatus(_T("Successfully sent DDE request."));
+            wxLogStatus(wxT("Successfully sent DDE request."));
         }
     }
 }
         }
     }
 }
@@ -912,16 +1275,54 @@ void MyFrame::OnIdle(wxIdleEvent& event)
     }
 }
 
     }
 }
 
-void MyFrame::OnTimer(wxTimerEvent& WXUNUSED(event))
+void MyFrame::OnIdleTimer(wxTimerEvent& WXUNUSED(event))
 {
     wxWakeUpIdle();
 }
 
 {
     wxWakeUpIdle();
 }
 
+void MyFrame::OnBgTimer(wxTimerEvent& WXUNUSED(event))
+{
+    static unsigned long s_ticks = 0;
+    SetStatusText(wxString::Format("%lu ticks", s_ticks++), 1);
+}
+
 void MyFrame::OnProcessTerminated(MyPipedProcess *process)
 {
 void MyFrame::OnProcessTerminated(MyPipedProcess *process)
 {
-    RemoveAsyncProcess(process);
+    RemovePipedProcess(process);
+}
+
+void MyFrame::OnAsyncTermination(MyProcess *process)
+{
+    m_allAsync.Remove(process);
+
+    delete process;
+}
+
+void MyFrame::AddPipedProcess(MyPipedProcess *process)
+{
+    if ( m_running.IsEmpty() )
+    {
+        // we want to start getting the timer events to ensure that a
+        // steady stream of idle events comes in -- otherwise we
+        // wouldn't be able to poll the child process input
+        m_timerIdleWakeUp.Start(100);
+    }
+    //else: the timer is already running
+
+    m_running.Add(process);
+    m_allAsync.Add(process);
 }
 
 }
 
+void MyFrame::RemovePipedProcess(MyPipedProcess *process)
+{
+    m_running.Remove(process);
+
+    if ( m_running.IsEmpty() )
+    {
+        // we don't need to get idle events all the time any more
+        m_timerIdleWakeUp.Stop();
+    }
+}
 
 void MyFrame::ShowOutput(const wxString& cmd,
                          const wxArrayString& output,
 
 void MyFrame::ShowOutput(const wxString& cmd,
                          const wxArrayString& output,
@@ -931,7 +1332,7 @@ void MyFrame::ShowOutput(const wxString& cmd,
     if ( !count )
         return;
 
     if ( !count )
         return;
 
-    m_lbox->Append(wxString::Format(_T("--- %s of '%s' ---"),
+    m_lbox->Append(wxString::Format(wxT("--- %s of '%s' ---"),
                                     title.c_str(), cmd.c_str()));
 
     for ( size_t n = 0; n < count; n++ )
                                     title.c_str(), cmd.c_str()));
 
     for ( size_t n = 0; n < count; n++ )
@@ -939,7 +1340,7 @@ void MyFrame::ShowOutput(const wxString& cmd,
         m_lbox->Append(output[n]);
     }
 
         m_lbox->Append(output[n]);
     }
 
-    m_lbox->Append(wxString::Format(_T("--- End of %s ---"),
+    m_lbox->Append(wxString::Format(wxT("--- End of %s ---"),
                                     title.Lower().c_str()));
 }
 
                                     title.Lower().c_str()));
 }
 
@@ -949,11 +1350,10 @@ void MyFrame::ShowOutput(const wxString& cmd,
 
 void MyProcess::OnTerminate(int pid, int status)
 {
 
 void MyProcess::OnTerminate(int pid, int status)
 {
-    wxLogStatus(m_parent, _T("Process %u ('%s') terminated with exit code %d."),
+    wxLogStatus(m_parent, wxT("Process %u ('%s') terminated with exit code %d."),
                 pid, m_cmd.c_str(), status);
 
                 pid, m_cmd.c_str(), status);
 
-    // we're not needed any more
-    delete this;
+    m_parent->OnAsyncTermination(this);
 }
 
 // ----------------------------------------------------------------------------
 }
 
 // ----------------------------------------------------------------------------
@@ -962,7 +1362,7 @@ void MyProcess::OnTerminate(int pid, int status)
 
 bool MyPipedProcess::HasInput()
 {
 
 bool MyPipedProcess::HasInput()
 {
-    bool hasInput = FALSE;
+    bool hasInput = false;
 
     if ( IsInputAvailable() )
     {
 
     if ( IsInputAvailable() )
     {
@@ -970,11 +1370,11 @@ bool MyPipedProcess::HasInput()
 
         // this assumes that the output is always line buffered
         wxString msg;
 
         // this assumes that the output is always line buffered
         wxString msg;
-        msg << m_cmd << _T(" (stdout): ") << tis.ReadLine();
+        msg << m_cmd << wxT(" (stdout): ") << tis.ReadLine();
 
         m_parent->GetLogListBox()->Append(msg);
 
 
         m_parent->GetLogListBox()->Append(msg);
 
-        hasInput = TRUE;
+        hasInput = true;
     }
 
     if ( IsErrorAvailable() )
     }
 
     if ( IsErrorAvailable() )
@@ -983,11 +1383,11 @@ bool MyPipedProcess::HasInput()
 
         // this assumes that the output is always line buffered
         wxString msg;
 
         // this assumes that the output is always line buffered
         wxString msg;
-        msg << m_cmd << _T(" (stderr): ") << tis.ReadLine();
+        msg << m_cmd << wxT(" (stderr): ") << tis.ReadLine();
 
         m_parent->GetLogListBox()->Append(msg);
 
 
         m_parent->GetLogListBox()->Append(msg);
 
-        hasInput = TRUE;
+        hasInput = true;
     }
 
     return hasInput;
     }
 
     return hasInput;
@@ -1010,7 +1410,7 @@ void MyPipedProcess::OnTerminate(int pid, int status)
 
 bool MyPipedProcess2::HasInput()
 {
 
 bool MyPipedProcess2::HasInput()
 {
-    if ( !!m_input )
+    if ( !m_input.empty() )
     {
         wxTextOutputStream os(*GetOutputStream());
         os.WriteString(m_input);
     {
         wxTextOutputStream os(*GetOutputStream());
         os.WriteString(m_input);
@@ -1019,7 +1419,7 @@ bool MyPipedProcess2::HasInput()
         m_input.clear();
 
         // call us once again - may be we'll have output
         m_input.clear();
 
         // call us once again - may be we'll have output
-        return TRUE;
+        return true;
     }
 
     return MyPipedProcess::HasInput();
     }
 
     return MyPipedProcess::HasInput();
@@ -1032,33 +1432,45 @@ bool MyPipedProcess2::HasInput()
 MyPipeFrame::MyPipeFrame(wxFrame *parent,
                          const wxString& cmd,
                          wxProcess *process)
 MyPipeFrame::MyPipeFrame(wxFrame *parent,
                          const wxString& cmd,
                          wxProcess *process)
-           : wxFrame(parent, -1, cmd),
+           : wxFrame(parent, wxID_ANY, cmd),
              m_process(process),
              // in a real program we'd check that the streams are !NULL here
              m_process(process),
              // in a real program we'd check that the streams are !NULL here
+             m_out(*process->GetOutputStream()),
              m_in(*process->GetInputStream()),
              m_in(*process->GetInputStream()),
-             m_out(*process->GetOutputStream())
+             m_err(*process->GetErrorStream())
 {
     m_process->SetNextHandler(this);
 
 {
     m_process->SetNextHandler(this);
 
-    wxPanel *panel = new wxPanel(this, -1);
+    wxPanel *panel = new wxPanel(this, wxID_ANY);
 
 
-    m_textIn = new wxTextCtrl(panel, -1, _T(""),
+    m_textOut = new wxTextCtrl(panel, wxID_ANY, wxEmptyString,
                               wxDefaultPosition, wxDefaultSize,
                               wxTE_PROCESS_ENTER);
                               wxDefaultPosition, wxDefaultSize,
                               wxTE_PROCESS_ENTER);
-    m_textOut = new wxTextCtrl(panel, -1, _T(""));
-    m_textOut->SetEditable(FALSE);
+    m_textIn = new wxTextCtrl(panel, wxID_ANY, wxEmptyString,
+                               wxDefaultPosition, wxDefaultSize,
+                               wxTE_MULTILINE | wxTE_RICH);
+    m_textIn->SetEditable(false);
+    m_textErr = new wxTextCtrl(panel, wxID_ANY, wxEmptyString,
+                               wxDefaultPosition, wxDefaultSize,
+                               wxTE_MULTILINE | wxTE_RICH);
+    m_textErr->SetEditable(false);
 
     wxSizer *sizerTop = new wxBoxSizer(wxVERTICAL);
 
     wxSizer *sizerTop = new wxBoxSizer(wxVERTICAL);
-    sizerTop->Add(m_textIn, 0, wxGROW | wxALL, 5);
+    sizerTop->Add(m_textOut, 0, wxGROW | wxALL, 5);
 
     wxSizer *sizerBtns = new wxBoxSizer(wxHORIZONTAL);
 
     wxSizer *sizerBtns = new wxBoxSizer(wxHORIZONTAL);
-    sizerBtns->Add(new wxButton(panel, Exec_Btn_Send, _T("&Send")), 0,
-                   wxALL, 10);
-    sizerBtns->Add(new wxButton(panel, Exec_Btn_Get, _T("&Get")), 0,
-                   wxALL, 10);
+    sizerBtns->
+        Add(new wxButton(panel, Exec_Btn_Send, wxT("&Send")), 0, wxALL, 5);
+    sizerBtns->
+        Add(new wxButton(panel, Exec_Btn_SendFile, wxT("&File...")), 0, wxALL, 5);
+    sizerBtns->
+        Add(new wxButton(panel, Exec_Btn_Get, wxT("&Get")), 0, wxALL, 5);
+    sizerBtns->
+        Add(new wxButton(panel, Exec_Btn_Close, wxT("&Close")), 0, wxALL, 5);
 
     sizerTop->Add(sizerBtns, 0, wxCENTRE | wxALL, 5);
 
     sizerTop->Add(sizerBtns, 0, wxCENTRE | wxALL, 5);
-    sizerTop->Add(m_textOut, 0, wxGROW | wxALL, 5);
+    sizerTop->Add(m_textIn, 1, wxGROW | wxALL, 5);
+    sizerTop->Add(m_textErr, 1, wxGROW | wxALL, 5);
 
     panel->SetSizer(sizerTop);
     sizerTop->Fit(this);
 
     panel->SetSizer(sizerTop);
     sizerTop->Fit(this);
@@ -1066,18 +1478,76 @@ MyPipeFrame::MyPipeFrame(wxFrame *parent,
     Show();
 }
 
     Show();
 }
 
+void MyPipeFrame::OnBtnSendFile(wxCommandEvent& WXUNUSED(event))
+{
+#if wxUSE_FILEDLG
+    wxFileDialog filedlg(this, wxT("Select file to send"));
+    if ( filedlg.ShowModal() != wxID_OK )
+        return;
+
+    wxFFile file(filedlg.GetFilename(), wxT("r"));
+    wxString data;
+    if ( !file.IsOpened() || !file.ReadAll(&data) )
+        return;
+
+    // can't write the entire string at once, this risk overflowing the pipe
+    // and we would dead lock
+    size_t len = data.length();
+    const wxChar *pc = data.c_str();
+    while ( len )
+    {
+        const size_t CHUNK_SIZE = 4096;
+        m_out.Write(pc, len > CHUNK_SIZE ? CHUNK_SIZE : len);
+
+        // note that not all data could have been written as we don't block on
+        // the write end of the pipe
+        const size_t lenChunk = m_out.LastWrite();
+
+        pc += lenChunk;
+        len -= lenChunk;
+
+        DoGet();
+    }
+#endif // wxUSE_FILEDLG
+}
+
 void MyPipeFrame::DoGet()
 {
     // we don't have any way to be notified when any input appears on the
     // stream so we have to poll it :-(
 void MyPipeFrame::DoGet()
 {
     // we don't have any way to be notified when any input appears on the
     // stream so we have to poll it :-(
-    //
-    // NB: this really must be done because otherwise the other program might
-    //     not have enough time to receive or process our data and we'd read
-    //     an empty string
-    while ( !m_process->IsInputAvailable() && m_process->IsInputOpened() )
-        ;
+    DoGetFromStream(m_textIn, m_in);
+    DoGetFromStream(m_textErr, m_err);
+}
 
 
-    m_textOut->SetValue(m_in.ReadLine());
+void MyPipeFrame::DoGetFromStream(wxTextCtrl *text, wxInputStream& in)
+{
+    while ( in.CanRead() )
+    {
+        char buffer[4096];
+        buffer[in.Read(buffer, WXSIZEOF(buffer) - 1).LastRead()] = '\0';
+
+        text->AppendText(buffer);
+    }
+}
+
+void MyPipeFrame::DoClose()
+{
+    m_process->CloseOutput();
+
+    DisableInput();
+}
+
+void MyPipeFrame::DisableInput()
+{
+    m_textOut->SetEditable(false);
+    FindWindow(Exec_Btn_Send)->Disable();
+    FindWindow(Exec_Btn_SendFile)->Disable();
+    FindWindow(Exec_Btn_Close)->Disable();
+}
+
+void MyPipeFrame::DisableOutput()
+{
+    FindWindow(Exec_Btn_Get)->Disable();
 }
 
 void MyPipeFrame::OnClose(wxCloseEvent& event)
 }
 
 void MyPipeFrame::OnClose(wxCloseEvent& event)
@@ -1098,10 +1568,12 @@ void MyPipeFrame::OnClose(wxCloseEvent& event)
 
 void MyPipeFrame::OnProcessTerm(wxProcessEvent& WXUNUSED(event))
 {
 
 void MyPipeFrame::OnProcessTerm(wxProcessEvent& WXUNUSED(event))
 {
-    delete m_process;
-    m_process = NULL;
+    DoGet();
+
+    wxDELETE(m_process);
 
 
-    wxLogWarning(_T("The other process has terminated, closing"));
+    wxLogWarning(wxT("The other process has terminated, closing"));
 
 
-    Close();
+    DisableInput();
+    DisableOutput();
 }
 }