diff options
| author | Maksim Denisov <denisov.maksim@huawei.com> | 2024-10-04 13:52:55 +0200 |
|---|---|---|
| committer | Maksim Denisov <denisov.maksim@huawei.com> | 2024-10-07 17:51:22 +0200 |
| commit | 6d484db9b293aac4b8bd790eb7aead3a806f8c7b (patch) | |
| tree | c7a93cef8a95006f217d557300151ce5e8735437 /src/buildtool/system/system_command.hpp | |
| parent | 0fc0d9118b5e525fa582de0286c8e83bb0da4e16 (diff) | |
| download | justbuild-6d484db9b293aac4b8bd790eb7aead3a806f8c7b.tar.gz | |
Enable cppcoreguidelines-* checks.
Diffstat (limited to 'src/buildtool/system/system_command.hpp')
| -rw-r--r-- | src/buildtool/system/system_command.hpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/buildtool/system/system_command.hpp b/src/buildtool/system/system_command.hpp index ade2f54b..909c047a 100644 --- a/src/buildtool/system/system_command.hpp +++ b/src/buildtool/system/system_command.hpp @@ -201,7 +201,7 @@ class SystemCommand { // wait for child to finish and obtain return value int status{}; std::optional<int> retval{std::nullopt}; - do { + while (not retval) { if (::waitpid(pid, &status, 0) == -1) { // this should never happen logger_.Emit(LogLevel::Error, @@ -221,7 +221,7 @@ class SystemCommand { LogLevel::Debug, "Child got killed by signal {}", sig); } // continue waitpid() in case we got STOPSIG from child - } while (not retval); + } return retval; } |