Use nullptr instead of 0 for pointers

[libs/core.git] / source / time / timer.h

diff --git a/source/time/timer.h b/source/time/timer.h
index 21aeb2b4b5b02ea0b41932f4a3311028f35c7e55..33257319eba3466d89f28c650efcd26dde0a996d 100644 (file)
--- a/source/time/timer.h
+++ b/source/time/timer.h
@@ -4,6 +4,7 @@
 #include <vector>
 #include <sigc++/sigc++.h>
 #include <msp/core/mutex.h>
+#include <msp/core/noncopyable.h>
 #include <msp/core/semaphore.h>
 #include "timedelta.h"
 #include "timestamp.h"
@@ -18,7 +19,7 @@ of the returned slot.
 
 This class is thread-safe, to allow running timers in a separate thread.
 */
-class Timer
+class Timer: private NonCopyable
 {
 public:
        class Slot
@@ -40,7 +41,7 @@ public:
 private:
        struct SlotProxy
        {
-               Slot *slot;
+               Slot *slot = nullptr;
 
                SlotProxy(Slot *);
                bool operator<(const SlotProxy &) const;
@@ -49,7 +50,7 @@ private:
        std::vector<SlotProxy> slots;
        Semaphore sem;
        Mutex mutex;
-       bool blocking;
+       bool blocking = false;
 
 public:
        Timer();
@@ -66,9 +67,6 @@ public:
        /** Cancels a previously added timer. */
        void cancel(Slot &);
 
-       /** Deprecated.  Use one of the other overloads. */
-       void tick(bool block);
-
        /** Waits until a timer expires, then executes it.  If no timers have been
        set, blocks until one is added from another thread. */
        void tick();