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

1 files changed, 2 insertions, 8 deletions

diff --git a/src/utils/timer.h b/src/utils/timer.h
index 87c94a28..2c6ca891 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;