From: Adeel Kazmi Date: Thu, 24 Nov 2016 17:43:40 +0000 (+0000) Subject: Change Internal::Control destructor to protected X-Git-Tag: dali_1.2.17~12^2 X-Git-Url: http://review.tizen.org/git/?p=platform%2Fcore%2Fuifw%2Fdali-toolkit.git;a=commitdiff_plain;h=c8c89602dd8734c4fba423047b6a07c7c005b423;hp=9217ec0c1d483607ed6c8f08a1d9c238657ba5d6 Change Internal::Control destructor to protected This is a ref-counted object so another object should not be able to delete it. Change-Id: I05d2d262f509380586584acfa1fb6646fe1ba688 --- diff --git a/dali-toolkit/public-api/controls/control-impl.cpp b/dali-toolkit/public-api/controls/control-impl.cpp index b8d59eb..2bc13be 100644 --- a/dali-toolkit/public-api/controls/control-impl.cpp +++ b/dali-toolkit/public-api/controls/control-impl.cpp @@ -542,11 +542,6 @@ Toolkit::Control Control::New() return handle; } -Control::~Control() -{ - delete mImpl; -} - void Control::SetStyleName( const std::string& styleName ) { if( styleName != mImpl->mStyleName ) @@ -994,6 +989,11 @@ Control::Control( ControlBehaviour behaviourFlags ) mImpl->mFlags = behaviourFlags; } +Control::~Control() +{ + delete mImpl; +} + void Control::Initialize() { // Call deriving classes so initialised before styling is applied to them. diff --git a/dali-toolkit/public-api/controls/control-impl.h b/dali-toolkit/public-api/controls/control-impl.h index 9993237..95ea152 100644 --- a/dali-toolkit/public-api/controls/control-impl.h +++ b/dali-toolkit/public-api/controls/control-impl.h @@ -76,12 +76,6 @@ public: */ static Toolkit::Control New(); - /** - * @brief Virtual destructor. - * @SINCE_1_0.0 - */ - virtual ~Control(); - // Styling /** @@ -520,6 +514,12 @@ protected: // Helpers for deriving classes Control( ControlBehaviour behaviourFlags ); /** + * @brief Virtual destructor. + * @SINCE_1_0.0 + */ + virtual ~Control(); + + /** * @brief Second phase initialization. * @SINCE_1_0.0 */