From 3f9fdda7c6c6254a322afc2b09b33eba30a90f08 Mon Sep 17 00:00:00 2001 From: Maxim Mamontov Date: Wed, 14 Sep 2011 17:32:24 +0300 Subject: [PATCH 1/1] Hide copy constructor and assignement operator for ACTION Add virtual destructor for ACTIONS_LIST --- projects/stargazer/actions.h | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/projects/stargazer/actions.h b/projects/stargazer/actions.h index 53dde92b..6c1fc8a8 100644 --- a/projects/stargazer/actions.h +++ b/projects/stargazer/actions.h @@ -28,7 +28,7 @@ typedef void (ACTIVE_CLASS::*TYPE)(DATA_TYPE); class BASE_ACTION { public: - virtual ~BASE_ACTION() {}; + virtual ~BASE_ACTION() {} virtual void Invoke() = 0; }; @@ -44,6 +44,9 @@ public: : activeClass(ac), actor(a), data(d) {}; void Invoke(); private: + ACTION(const ACTION & rvalue); + ACTION & operator=(const ACTION & rvalue); + ACTIVE_CLASS & activeClass; typename ACTOR::TYPE actor; DATA_TYPE data; @@ -60,7 +63,7 @@ public: // Initialize mutex ACTIONS_LIST(); // Delete actions and destroy mutex - ~ACTIONS_LIST(); + virtual ~ACTIONS_LIST(); parent::iterator begin(); parent::iterator end(); -- 2.44.2