diff options
| author | Karen Arutyunov <karen@codesynthesis.com> | 2016-11-03 15:58:11 +0300 |
|---|---|---|
| committer | Boris Kolpackov <boris@codesynthesis.com> | 2016-11-04 09:26:37 +0200 |
| commit | 57c7577fa91ffc024e5c696a91abe19f5547a787 (patch) | |
| tree | 410a09a2d1ebf6da5e770b644c7a1806f2d1aef9 /build2/test/script/runner.cxx | |
| parent | d933699b4dd106c227f8d07a2471d5f39f1c82af (diff) | |
Fix Clang and VC warnings
Diffstat (limited to 'build2/test/script/runner.cxx')
| -rw-r--r-- | build2/test/script/runner.cxx | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/build2/test/script/runner.cxx b/build2/test/script/runner.cxx index 29f7a37..b071cf3 100644 --- a/build2/test/script/runner.cxx +++ b/build2/test/script/runner.cxx @@ -526,7 +526,11 @@ namespace build2 // stderr (if cached), print the proper diagnostics and fail. // optional<process::status_type> status (move (pr.status)); - bool valid_status (status && *status >= 0 && *status < 256); + + // Comparison *status >= 0 causes "always true" warning on Windows + // where process::status_type is defined as uint32_t. + // + bool valid_status (status && *status + 1 > 0 && *status < 256); bool eq (c.exit.comparison == exit_comparison::eq); bool correct_status (valid_status && |