TheCloud/cpp-subprocess
1
0
Fork 0
mirror of https://github.com/arun11299/cpp-subprocess.git synced 2025-08-04 12:26:19 -04:00
Code Issues Packages Projects Releases Wiki Activity

msvc: Fix "warning C4101: 'status': unreferenced local variable"

Browse Source 
This change is required to build downstream projects using MSVC with
the `/WX` compiler option.

No behavior changes.
This commit is contained in:
Hennadii Stepanov 2023-11-30 15:58:08 +00:00
parent d809374661
commit 1bed18f5b7
No known key found for this signature in database
GPG Key ID: 410108112E7EA81F
1 changed files with 6 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
subprocess.hpp
View File

@ -1427,8 +1427,6 @@ inline int Popen::wait() noexcept(false)
inline int Popen::poll() noexcept(false)
{
int status;
#ifndef _MSC_VER
if (!child_created_) return -1; // TODO: ??
#endif
@ -1436,13 +1434,7 @@ inline int Popen::poll() noexcept(false)
#ifdef __USING_WINDOWS__
int ret = WaitForSingleObject(process_handle_, 0);
if (ret != WAIT_OBJECT_0) return -1;
#else
// Returns zero if child is still running
int ret = waitpid(child_pid_, &status, WNOHANG);
if (ret == 0) return -1;
#endif
#ifdef __USING_WINDOWS__
DWORD dretcode_;
if (FALSE == GetExitCodeProcess(process_handle_, &dretcode_))
throw OSError("GetExitCodeProcess", 0);
@ -1452,6 +1444,12 @@ inline int Popen::poll() noexcept(false)
return retcode_;
#else
int status;
// Returns zero if child is still running
int ret = waitpid(child_pid_, &status, WNOHANG);
if (ret == 0) return -1;
if (ret == child_pid_) {
if (WIFSIGNALED(status)) {
retcode_ = WTERMSIG(status);