Proper silencing for the switch-case fall-through warning
ClosedPublic

Authored by TallFurryMan on Aug 29 2018, 7:10 AM.

Details

Summary

This changeset silences the gcc warning for the case fall-through.
No idea if this will work without warning for msvc, but that doesn't change much compared to the previous state.
C++17 can use "fallthrough;" to specify an execution path that intentionally falls through the following case.
Q_FALLTHROUGH was introduced in QT5.8, thus can't be used for Raspbian, still using QT5.4.

Test Plan

Without this change, gcc issues a warning on the case fall-through in function unParkMount.
With this change fall-through is acknlowledged and the warning doesn't appear.

Diff Detail

Repository
R321 KStars
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
TallFurryMan requested review of this revision.Aug 29 2018, 7:10 AM
TallFurryMan created this revision.
mutlaqja accepted this revision.Aug 29 2018, 8:38 AM
This revision is now accepted and ready to land.Aug 29 2018, 8:38 AM
This revision was automatically updated to reflect the committed changes.