Small cleanups in utils::Timer

* Not nice to pass boolean parameter to constructor for preventing the
  timer from running on creation. Just call 'start' on it explicitly.
* getTimeInMillisec could be made static.

Reviewed-by: Yohann Ferreira

2 files changed, 17 insertions, 20 deletions

diff --git a/src/utils/timer.cpp b/src/utils/timer.cpp
index a79a546..aea10ad 100644
--- a/src/utils/timer.cpp
+++ b/src/utils/timer.cpp
@@ -27,12 +27,25 @@
 #include <windows.h>
 #endif
 
+/**
+ * Calls gettimeofday() and converts it into milliseconds.
+ */
+static uint64_t getTimeInMillisec()
+{
+    uint64_t timeInMillisec;
+    timeval time;
+
+    gettimeofday(&time, 0);
+    timeInMillisec = (uint64_t)time.tv_sec * 1000 + time.tv_usec / 1000;
+    return timeInMillisec;
+}
+
 namespace utils
 {
 
-Timer::Timer(unsigned int ms, bool createActive)
+Timer::Timer(unsigned int ms)
 {
-    active = createActive;
+    active = false;
     interval = ms;
     lastpulse = getTimeInMillisec();
 }
@@ -89,14 +102,4 @@ void Timer::changeInterval(unsigned int newinterval)
     interval = newinterval;
 }
 
-uint64_t Timer::getTimeInMillisec()
-{
-    uint64_t timeInMillisec;
-    timeval time;
-
-    gettimeofday(&time, 0);
-    timeInMillisec = (uint64_t)time.tv_sec * 1000 + time.tv_usec / 1000;
-    return timeInMillisec;
-}
-
 } // ::utils
diff --git a/src/utils/timer.h b/src/utils/timer.h
index 87c94a2..2c6ca89 100644
--- a/src/utils/timer.h
+++ b/src/utils/timer.h
@@ -41,9 +41,8 @@ class Timer
          * Constructor.
          *
          * @param ms           the interval in milliseconds
-         * @param createActive whether the timer should be implicitly started
          */
-        Timer(unsigned int ms, bool createActive = true);
+        Timer(unsigned int ms);
 
         /**
          * Returns the number of elapsed ticks since last call.
@@ -68,15 +67,10 @@ class Timer
         /**
          * Changes the interval between two pulses.
          */
-        void changeInterval (unsigned int newinterval);
+        void changeInterval(unsigned int newinterval);
 
     private:
         /**
-         * Calls gettimeofday() and converts it into milliseconds.
-         */
-        uint64_t getTimeInMillisec();
-
-        /**
          * Interval between two pulses.
          */
         unsigned int interval;