[util] Use HB_FALLTHROUGH
authorBehdad Esfahbod <behdad@behdad.org>
Sun, 30 Sep 2018 09:49:08 +0000 (05:49 -0400)
committerBehdad Esfahbod <behdad@behdad.org>
Sun, 30 Sep 2018 10:08:11 +0000 (06:08 -0400)
commit9caa432d0c5c09c8151cfce1e2cc184fbdd89594
tree58c2082eef17c23585032b3c1299ed70e11429b6
parent2e728a7d86c714d845524a0722c2b653feb9d915
[util] Use HB_FALLTHROUGH

Sure, gcc knows to warn about this as well:

../../util/options.cc:175:17: warning: this statement may fall through [-Wimplicit-fallthrough=]
     case 1: m.r = m.t;
             ~~~~^~~~~
../../util/options.cc:176:5: note: here
     case 2: m.b = m.t;
     ^~~~

But HOLY SMOKES, look at clang -Weverything bot message:

options.cc:176:5: warning: unannotated fall-through between switch labels [-Wimplicit-fallthrough]
    case 2: m.b = m.t;
    ^
options.cc:176:5: note: insert 'HB_FALLTHROUGH;' to silence this warning
    case 2: m.b = m.t;
    ^
    HB_FALLTHROUGH;

Right, it's telling me to insert "HB_FALLTHROUGH;" there!!!!!!!!!
util/options.cc