diff options
author | Pasi Keränen <pasi.keranen@qt.io> | 2018-09-10 13:49:54 +0300 |
---|---|---|
committer | Pasi Keränen <pasi.keranen@qt.io> | 2018-09-11 05:45:19 +0000 |
commit | 627879ea6e025c8335727867ef50a3b13183d7a0 (patch) | |
tree | b412e27620440c785617c6c991b28296be7f3441 /src/Authoring/Studio/Palettes/Action/ActionView.cpp | |
parent | 1880495b000c3bc4eac819120c2dccfb03e4ce52 (diff) |
Fix switch-cases not handling all possible values
Add default handling for switch cases missing it and remove the switch
case related warning suppression with clang builds.
Task-number: QT3DS-2243
Change-Id: Ib87d51468bbfa00284c6849363751ccb056f7eb7
Reviewed-by: Miikka Heikkinen <miikka.heikkinen@qt.io>
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>
Diffstat (limited to 'src/Authoring/Studio/Palettes/Action/ActionView.cpp')
-rw-r--r-- | src/Authoring/Studio/Palettes/Action/ActionView.cpp | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/Authoring/Studio/Palettes/Action/ActionView.cpp b/src/Authoring/Studio/Palettes/Action/ActionView.cpp index 1cb5ab2d..dc3e6ff1 100644 --- a/src/Authoring/Studio/Palettes/Action/ActionView.cpp +++ b/src/Authoring/Studio/Palettes/Action/ActionView.cpp @@ -728,7 +728,11 @@ void ActionView::OnSelectionSet(Q3DStudio::SSelectedValue inSelectable) Q_ASSERT(GuidValid(theComponentGuid)); theInstance = theBridge->GetInstanceByGUID(theComponentGuid); Q_ASSERT(theInstance.Valid()); - } break; + } + break; + default: + // Do nothing on slide insertion, guide and unknown types + return; }; setItem(theInstance); |