Use symbolic WAIT_FAILED constant instead of raw -1.

No real changes, just make the code slightly more readable by using a symbolic
constant instead of a magic value.

This also avoids g++ warnings about implicit conversion of a signed value (-1)
to unsigned type (DWORD).

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

diff --git a/src/msw/utilsexc.cpp b/src/msw/utilsexc.cpp
index 6fc63c0ee1f7b22dca338f66e11afc58cf7a8601..2887ce5340db00c899e2c448fa181aa8eeee3765 100644 (file)
--- a/src/msw/utilsexc.cpp
+++ b/src/msw/utilsexc.cpp
@@ -1010,7 +1010,7 @@ long wxExecute(const wxString& cmd, int flags, wxProcess *handler,
                 wxFAIL_MSG( wxT("unexpected WaitForInputIdle() return code") );
                 // fall through
 
                 wxFAIL_MSG( wxT("unexpected WaitForInputIdle() return code") );
                 // fall through
 

-            case -1:
+            case WAIT_FAILED:

                 wxLogLastError(wxT("WaitForInputIdle() in wxExecute"));
 
             case WAIT_TIMEOUT:
                 wxLogLastError(wxT("WaitForInputIdle() in wxExecute"));
 
             case WAIT_TIMEOUT: