You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@activemq.apache.org by ta...@apache.org on 2012/11/20 23:27:17 UTC

svn commit: r1411911 - /activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/concurrent/ThreadPoolExecutor.cpp

Author: tabish
Date: Tue Nov 20 22:27:16 2012
New Revision: 1411911

URL: http://svn.apache.org/viewvc?rev=1411911&view=rev
Log:
Fix for destructor that should be declared virtual 

Modified:
    activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/concurrent/ThreadPoolExecutor.cpp

Modified: activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/concurrent/ThreadPoolExecutor.cpp
URL: http://svn.apache.org/viewvc/activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/concurrent/ThreadPoolExecutor.cpp?rev=1411911&r1=1411910&r2=1411911&view=diff
==============================================================================
--- activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/concurrent/ThreadPoolExecutor.cpp (original)
+++ activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/concurrent/ThreadPoolExecutor.cpp Tue Nov 20 22:27:16 2012
@@ -154,8 +154,8 @@ namespace concurrent{
 
         private:
 
-            Worker( const Worker& );
-            Worker& operator= ( const Worker& );
+            Worker(const Worker&);
+            Worker& operator=(const Worker&);
 
          public:
 
@@ -170,7 +170,7 @@ namespace concurrent{
                 this->thread.reset(kernel->factory->newThread(this));
             }
 
-            ~Worker() {}
+            virtual ~Worker() {}
 
             void run() {
                 // Delegate the running of this task to the Kernel so that all the logic