From d970b802a4605577b93deff523c94982a832d37b Mon Sep 17 00:00:00 2001 From: Vadim Zeitlin Date: Mon, 12 Dec 2016 23:20:08 +0100 Subject: [PATCH] Improve confusing comment in wxMSW wxExecute() implementation The comment was difficult to understand because of the missing "in" preposition, so add it and extend the comment further for clarity. See https://github.com/wxWidgets/wxWidgets/pull/369 --- src/msw/utilsexc.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/msw/utilsexc.cpp b/src/msw/utilsexc.cpp index 3151ba42bd..43a88d98af 100644 --- a/src/msw/utilsexc.cpp +++ b/src/msw/utilsexc.cpp @@ -921,7 +921,9 @@ long wxExecute(const wxString& cmd, int flags, wxProcess *handler, if ( flags & wxEXEC_SYNC ) { // handler may be !NULL for capturing program output, but we don't use - // it wxExecuteData struct in this case + // it in wxExecuteData struct in this case because it's only needed + // there for calling OnTerminate() on it and we don't do this when + // executing synchronously data->handler = NULL; } else