summaryrefslogtreecommitdiffstats
path: root/src/Authoring/Studio/Palettes/Action/ActionView.cpp
diff options
context:
space:
mode:
authorPasi Keränen <pasi.keranen@qt.io>2018-09-10 13:49:54 +0300
committerPasi Keränen <pasi.keranen@qt.io>2018-09-11 05:45:19 +0000
commit627879ea6e025c8335727867ef50a3b13183d7a0 (patch)
treeb412e27620440c785617c6c991b28296be7f3441 /src/Authoring/Studio/Palettes/Action/ActionView.cpp
parent1880495b000c3bc4eac819120c2dccfb03e4ce52 (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.cpp6
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);