Qt has its own macro to tell the compiler that the lack of the break
statement is intentional. One of the advantages of using Q_FALLTHROUGH
is that it explicitly says other developers that the break statement is
missing on purpose. Another advantage is that we can use directives,
see https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
Details
Details
- Reviewers
davidedmundson - Group Reviewers
Breeze - Commits
- R31:2983e63983cd: [kstyle] use Q_FALLTHROUGH in breezesplitterproxy
Compiled Breeze with GCC 7 and Clang, no "this statement may fall
through" warnings whatsoever.
Diff Detail
Diff Detail
- Repository
- R31 Breeze
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.