[kstyle] use Q_FALLTHROUGH in breezesplitterproxy

Authored by zzag on Apr 1 2018, 8:45 AM.

Description

[kstyle] use Q_FALLTHROUGH in breezesplitterproxy

Summary:
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

Test Plan:
Compiled Breeze with GCC 7 and Clang, no "this statement may fall
through" warnings whatsoever.

Reviewers: Breeze, davidedmundson

Reviewed By: davidedmundson

Subscribers: plasma-devel

Tags: Plasma

Differential Revision: https://phabricator.kde.org/D11860

Details

Committed
zzagApr 3 2018, 12:20 PM
Reviewer
davidedmundson
Differential Revision
D11860: [kstyle] use Q_FALLTHROUGH in breezesplitterproxy
Parents
R31:7821b4789593: [kdecoration] delete unused m_buttons field
Branches
Unknown
Tags
Unknown