From bd149b909feb037a09f9c23c0c55f4f8df61e74c Mon Sep 17 00:00:00 2001 From: elsid Date: Sun, 17 Nov 2019 16:37:38 +0100 Subject: [PATCH] Remove unused virtual modifiers in ShadowManager /home/elsid/dev/openmw/components/sceneutil/shadow.cpp:100:9: warning: Call to virtual function during construction [clang-analyzer-optin.cplusplus.VirtualCall] setupShadowSettings(); ^ /home/elsid/dev/openmw/components/sceneutil/shadow.cpp:100:9: note: This constructor of an object of type 'ShadowManager' has not returned when the virtual method was called /home/elsid/dev/openmw/components/sceneutil/shadow.cpp:100:9: note: Call to virtual function during construction /home/elsid/dev/openmw/components/sceneutil/shadow.cpp:104:9: warning: Call to virtual function during construction [clang-analyzer-optin.cplusplus.VirtualCall] enableOutdoorMode(); ^ /home/elsid/dev/openmw/components/sceneutil/shadow.cpp:104:9: note: This constructor of an object of type 'ShadowManager' has not returned when the virtual method was called /home/elsid/dev/openmw/components/sceneutil/shadow.cpp:104:9: note: Call to virtual function during construction --- components/sceneutil/shadow.hpp | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/components/sceneutil/shadow.hpp b/components/sceneutil/shadow.hpp index 928de4543a..c823ecf860 100644 --- a/components/sceneutil/shadow.hpp +++ b/components/sceneutil/shadow.hpp @@ -19,15 +19,13 @@ namespace SceneUtil ShadowManager(osg::ref_ptr sceneRoot, osg::ref_ptr rootNode, unsigned int outdoorShadowCastingMask, unsigned int indoorShadowCastingMask, Shader::ShaderManager &shaderManager); - virtual ~ShadowManager() = default; + void setupShadowSettings(); - virtual void setupShadowSettings(); + Shader::ShaderManager::DefineMap getShadowDefines(); - virtual Shader::ShaderManager::DefineMap getShadowDefines(); + void enableIndoorMode(); - virtual void enableIndoorMode(); - - virtual void enableOutdoorMode(); + void enableOutdoorMode(); protected: bool mEnableShadows;