Trivial: Add explicit [[fallthrough]] to switch statement #15973

pull HenryYoung42 wants to merge 1 commits into bitcoin:master from HenryYoung42:master changing 1 files +2 −0
  1. HenryYoung42 commented at 2:39 PM on May 7, 2019: none

    This avoids compiler warning "this statement may fall through" which would be generated if -Wimplicit-fallthrough were enabled.

  2. Trivial: Add explicit [[fallthrough]] to switch statement
    This avoids compiler warning "this statement may fall through" which would be generated if -Wimplicit-fallthrough were enabled.
    da07d85aba
  3. fanquake added the label Refactoring on May 7, 2019
  4. promag commented at 2:44 PM on May 7, 2019: member

    This is only available from C++17.

  5. HenryYoung42 commented at 2:51 PM on May 7, 2019: none

    Ah then that explains why the Bitcoin build is not seeing this warning, whereas a forked project is. I thought perhaps the warning was disabled but did not check first - my bad. Any idea when Bitcoin will move up to C++17 ?

  6. promag commented at 2:57 PM on May 7, 2019: member

    I don't know, I guess there's no rush.

  7. MarcoFalke commented at 3:06 PM on May 7, 2019: member

    Yeah, this should be revisited after c++17, which is going to happen after the next major release earliest.

  8. MarcoFalke closed this on May 7, 2019

  9. MarcoFalke locked this on Dec 16, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-22 06:14 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me