diff options
author | Boris Kolpackov <boris@codesynthesis.com> | 2017-12-01 17:02:16 +0200 |
---|---|---|
committer | Boris Kolpackov <boris@codesynthesis.com> | 2017-12-01 17:02:16 +0200 |
commit | c49b0e8ca53928cc20e9d4832536baaeee6d7a9d (patch) | |
tree | 91c980addf8ddc0c04feebb93699488e4385918b /build2/algorithm.cxx | |
parent | febcacdb5a60d37c2a56c9aad7b636be799940cd (diff) |
Fix GCC 7 -fimplicit-fallthrough warnings
Diffstat (limited to 'build2/algorithm.cxx')
-rw-r--r-- | build2/algorithm.cxx | 21 |
1 files changed, 12 insertions, 9 deletions
diff --git a/build2/algorithm.cxx b/build2/algorithm.cxx index fc71c1a..a532d7c 100644 --- a/build2/algorithm.cxx +++ b/build2/algorithm.cxx @@ -499,8 +499,9 @@ namespace build2 if (try_match) return make_pair (false, target_state::unknown); - // Fall through (to issue diagnostics). + // To issue diagnostics ... } + // Fall through. case target::offset_touched: { // Match. @@ -521,9 +522,8 @@ namespace build2 // t.state_ is not yet set. // return make_pair (true, target_state::unknown); - - // Fall through. } + // Fall through. case target::offset_matched: { // Apply. @@ -653,14 +653,16 @@ namespace build2 if ((r = g.group_members (a)).members != nullptr) break; - // Fall through to apply. + // To apply ... } - // @@ Doing match without execute messes up our target_count. Does - // not seem like it will be easy to fix (we don't know whether - // someone else will execute this target). - // + // Fall through. case target::offset_matched: { + // @@ Doing match without execute messes up our target_count. Does + // not seem like it will be easy to fix (we don't know whether + // someone else will execute this target). + // + // Apply (locked). // if (match_impl (a, l, true).second == target_state::failed) @@ -669,10 +671,11 @@ namespace build2 if ((r = g.group_members (a)).members != nullptr) break; - // Unlock and fall through to execute. + // Unlock and to execute ... // l.unlock (); } + // Fall through. case target::offset_applied: { // Execute (unlocked). |