Merge pull request #7 from ivanmari/master

Thread safety. Thread safe flag, using positive logic.

Author: shalithasuranga

sample.cpp

@@ -4,7 +4,7 @@
 using namespace std;
 
 int main() {
-    Timer t = Timer();
+    Timer t;
 
     t.setInterval([&]() {
         cout << "Hey.. After each 1s..." << endl;
@@ -21,4 +21,4 @@ int main() {
 
 
     while(true); // Keep mail thread active
-}
+}

timercpp.h

@@ -1,10 +1,11 @@
 #include <iostream>
 #include <thread>
 #include <chrono>
+#include <atomic>
 
 class Timer {
-    bool clear = false;
-
+	std::atomic<bool> active{true};
+	
     public:
         void setTimeout(auto function, int delay);
         void setInterval(auto function, int interval);
@@ -13,29 +14,28 @@ class Timer {
 };
 
 void Timer::setTimeout(auto function, int delay) {
-    this->clear = false;
+    active = true;
     std::thread t([=]() {
-        if(this->clear) return;
+        if(!active.load()) return;
         std::this_thread::sleep_for(std::chrono::milliseconds(delay));
-        if(this->clear) return;
+        if(!active.load()) return;
         function();
     });
     t.detach();
 }
 
 void Timer::setInterval(auto function, int interval) {
-    this->clear = false;
+    active = true;
     std::thread t([=]() {
-        while(true) {
-            if(this->clear) return;
+        while(active.load()) {
             std::this_thread::sleep_for(std::chrono::milliseconds(interval));
-            if(this->clear) return;
+            if(!active.load()) return;
             function();
         }
     });
     t.detach();
 }
 
 void Timer::stop() {
-    this->clear = true;
-}
+    active = false;
+}