#30 small fix up on enable task

Author: dave

examples/longSchedule/longSchedule.ino

@@ -25,7 +25,7 @@ public:
 
     void exec() override {
         callCount++;
-        Serial.print("Called my task executable");
+        Serial.print("Called my task executable ");
         Serial.println(callCount);
 
         taskManager.setTaskEnabled(taskToSuspend, enableTask);
@@ -48,7 +48,9 @@ void setup() {
     Serial.begin(115200);
 
     Serial.println("Started long schedule example");
-    // First two long schedules are global variables. If you use references to a variable like this it must be global
+
+    // First two long schedules are global variables.
+    // IMPORTANT NOTE: If you use references to a variable like this THEY MUST BE GLOBAL
     taskManager.registerEvent(&hourAndHalfSchedule);
     taskManager.registerEvent(&onceADaySchedule);
 

src/TaskTypes.cpp

@@ -85,6 +85,8 @@ unsigned long TimerTask::microsFromNow() {
 void TimerTask::execute() {
     RunningState runningState(this);
 
+    if(!isEnabled()) return;
+
     auto execType = (ExecutionType) (executeMode & EXECTYPE_MASK);
     switch (execType) {
         case EXECTYPE_EVENT:

src/TmLongSchedule.cpp

@@ -18,7 +18,7 @@ uint32_t makeDaySchedule(int days, int hours) {
 }
 
 TmLongSchedule::TmLongSchedule(uint32_t milliScheduleNext, Executable* toExecute) : milliSchedule(milliScheduleNext),
-        theExecutable(toExecute), lastScheduleTime(0), isTimerFn(true) { }
+        theExecutable(toExecute), lastScheduleTime(0), isTimerFn(false) { }
 
 TmLongSchedule::TmLongSchedule(uint32_t milliScheduleNext, TimerFn toExecute) : milliSchedule(milliScheduleNext),
         fnCallback(toExecute), lastScheduleTime(0), isTimerFn(true) { }