Changeset 11018 for code/trunk/src/libraries/tools/Timer.cc

Timestamp:

Jan 2, 2016, 8:59:12 PM (8 years ago)

Author:

landauf

Message:

I think it's sufficient if Timer is a Listable. A timer is often used as a member object in other classes, so it's probably a bad idea to make it a Destroyable since it will neither be deleted with destroy() nor will it survive the lifespan of its parent even if a StrongPtr references it.

File:

• code/trunk/src/libraries/tools/Timer.cc (modified) (4 diffs)

code/trunk/src/libraries/tools/Timer.cc

@@ -104 +104 @@
     {
         CommandExecutor::execute(command);
-        timer->destroy();
+        delete timer;
         delaytimers.right.erase(timer);
     }
@@ -114 +114 @@
     {
         for (boost::bimap<unsigned int, Timer*>::left_map::iterator it = delaytimers.left.begin(); it != delaytimers.left.end(); ++it)
-            it->second->destroy();
+            delete it->second;
 
         delaytimers.clear();
@@ -127 +127 @@
         if (it != delaytimers.left.end())
         {
-            it->second->destroy();
+            delete it->second;
             delaytimers.left.erase(it);
         }
@@ -210 +210 @@
 
         if (temp)
-            this->destroy();
+            delete this;
     }