diff options
| author | Marc Mutz <marc.mutz@kdab.com> | 2016-10-10 21:02:16 +0200 |
|---|---|---|
| committer | Marc Mutz <marc.mutz@kdab.com> | 2016-10-12 09:08:38 +0000 |
| commit | de48fd192b7973c4849ec79bce4cd491b5e8550f (patch) | |
| tree | 18c05bb98b556babe80eef1bd41e282e8bb81cb6 /src/plugins/platforms/xcb/qxcbbackingstore.cpp | |
| parent | 00304a3d57b9ba33b6a253b8736cf7e15aeac5c3 (diff) | |
QToolBarAreaLayoutInfo: add missing break statements in switch in distance()
A fall-through here is logically non-sensical, because of symmetry (or
lack thereof). Thus, a break must have been intended.
Add it.
While we're at it, also replace the default case label with the
non-functional enum value QInternal::DockCount, so that -Wswitch can
warn us if ever there should be a new DockPosition.
Found independently by both GCC 7 and Coverity.
Coverity-Id: 11145
Coverity-Id: 11146
Coverity-Id: 11147
Change-Id: I6bb31c1517e40f0cb06ceaee5aeb6fa78b84a523
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
Diffstat (limited to 'src/plugins/platforms/xcb/qxcbbackingstore.cpp')
0 files changed, 0 insertions, 0 deletions