useful-esp8266-lib
diff --git a/‎examples/ButtonInterrupt/ButtonInterrupt.ino
Lines changed: 4 additions & 2 deletions b/‎examples/ButtonInterrupt/ButtonInterrupt.ino
Lines changed: 4 additions & 2 deletions
diff --git a/‎examples/ButtonInterrupt/ButtonTask.h
Lines changed: 3 additions & 0 deletions b/‎examples/ButtonInterrupt/ButtonTask.h
Lines changed: 3 additions & 0 deletions
diff --git a/‎src/Task.h
Lines changed: 15 additions & 0 deletions b/‎src/Task.h
Lines changed: 15 additions & 0 deletions
diff --git a/‎src/TaskManager.cpp
Lines changed: 4 additions & 0 deletions b/‎src/TaskManager.cpp
Lines changed: 4 additions & 0 deletions
diff --git a/‎src/TaskManager.h
Lines changed: 4 additions & 1 deletion b/‎src/TaskManager.h
Lines changed: 4 additions & 1 deletion
@@ -52,6 +52,8 @@ void OnBButtonChanged(ButtonState state);
 void OnUpdateTaskLedOn(uint32_t deltaTime);
 void OnUpdateTaskLedOff(uint32_t deltaTime);
 void OnAutoSleep(uint32_t deltaTime);
+void ISR_ATTR OnExtIntLow();
+void ISR_ATTR OnExtIntHigh();
 
 ButtonTask AButtonTask(OnAButtonChanged, AButtonPin); 
 ButtonTask BButtonTask(OnBButtonChanged, BButtonPin); 
@@ -92,7 +94,7 @@ void loop()
     taskManager.Loop();
 }
 
-void OnExtIntLow()
+void ISR_ATTR OnExtIntLow()
 {
 // required due to Mega Level to wake up
 // so we toggle it off when it happens and setup
@@ -101,7 +103,7 @@ void OnExtIntLow()
 	attachInterrupt(WakeInterruptNumber, OnExtIntHigh, HIGH);
 }
 
-void OnExtIntHigh()
+void ISR_ATTR OnExtIntHigh()
 {
 // required due to Mega Level to wake up
 // so we toggle it off when it happens and setup
 
@@ -103,6 +103,9 @@ class ButtonTask : public Task
                     _timer -= deltaTimeMs;
                 }
                 break;
+
+            case ButtonState_Released:
+                break;
             }
         }
     }
 
@@ -19,6 +19,21 @@ See GNU Lesser General Public License at <http://www.gnu.org/licenses/>.
 #define countof(a) (sizeof(a) / sizeof(a[0]))
 #endif
 
+// Arduino has no standard for attributing methods used for ISRs
+// even though some platforms require it, so to simplify the problem
+// for end users, this provides a standard ISR_ATTR
+#if !defined(ISR_ATTR)
+
+#if defined(ARDUINO_ARCH_ESP8266) 
+#define ISR_ATTR ICACHE_RAM_ATTR
+#elif defined(ARDUINO_ARCH_ESP32)
+#define ISR_ATTR ICACHE_RAM_ATTR
+#else
+#define ISR_ATTR
+#endif
+
+#endif // !defined(ISR_ATTR)
+
 // if you need finer timing resolution than a millisecond
 // then enable the below, but this will limit the max interval 
 // to just over 70 minutes
 
@@ -166,6 +166,10 @@ void TaskManager::EnterSleep(uint32_t microSeconds,
     {
         system_rtc_mem_write(RTC_MEM_SLEEP_ADDR, state, sizeofState);
     }
+    if (microSeconds == 0)
+    {
+        microSeconds = ESP.deepSleepMax();
+    }
     ESP.deepSleep(microSeconds, mode);
 }
 
 
@@ -42,14 +42,17 @@ class TaskManager
 
 #if defined(ARDUINO_ARCH_ESP8266)
     // must have GPIO16 tied to RST
-    void EnterSleep(uint32_t microSeconds,
+    void EnterSleep(uint32_t microSeconds = 0,
         void* state = NULL,
         uint16_t sizeofState = 0,
         WakeMode mode = WAKE_RF_DEFAULT);
     bool RestartedFromSleep(void* state = NULL,
         uint16_t sizeofState = 0 );
+#elif defined(ARDUINO_ARCH_ESP32)
+#error "ESP32 support for sleep not implemented yet"
 #elif defined(__arm__)
     // Arm support for sleep not implemented yet
+#error "Arm support for sleep not implemented yet"
 #elif defined(ARDUINO_ARCH_AVR)
     void EnterSleep(uint8_t sleepMode = SLEEP_MODE_PWR_DOWN);
 #endif
Original file line number	Diff line number	Diff line change
`@@ -52,6 +52,8 @@ void OnBButtonChanged(ButtonState state);`
`52`	`52`	`void OnUpdateTaskLedOn(uint32_t deltaTime);`
`53`	`53`	`void OnUpdateTaskLedOff(uint32_t deltaTime);`
`54`	`54`	`void OnAutoSleep(uint32_t deltaTime);`
	`55`	`+void ISR_ATTR OnExtIntLow();`
	`56`	`+void ISR_ATTR OnExtIntHigh();`
`55`	`57`
`56`	`58`	`ButtonTask AButtonTask(OnAButtonChanged, AButtonPin);`
`57`	`59`	`ButtonTask BButtonTask(OnBButtonChanged, BButtonPin);`
`@@ -92,7 +94,7 @@ void loop()`
`92`	`94`	`taskManager.Loop();`
`93`	`95`	`}`
`94`	`96`
`95`		`-void OnExtIntLow()`
	`97`	`+void ISR_ATTR OnExtIntLow()`
`96`	`98`	`{`
`97`	`99`	`// required due to Mega Level to wake up`
`98`	`100`	`// so we toggle it off when it happens and setup`
`@@ -101,7 +103,7 @@ void OnExtIntLow()`
`101`	`103`	`attachInterrupt(WakeInterruptNumber, OnExtIntHigh, HIGH);`
`102`	`104`	`}`
`103`	`105`
`104`		`-void OnExtIntHigh()`
	`106`	`+void ISR_ATTR OnExtIntHigh()`
`105`	`107`	`{`
`106`	`108`	`// required due to Mega Level to wake up`
`107`	`109`	`// so we toggle it off when it happens and setup`
Original file line number	Diff line number	Diff line change
`@@ -103,6 +103,9 @@ class ButtonTask : public Task`
`103`	`103`	`_timer -= deltaTimeMs;`
`104`	`104`	`}`
`105`	`105`	`break;`
	`106`	`+`
	`107`	`+ case ButtonState_Released:`
	`108`	`+ break;`
`106`	`109`	`}`
`107`	`110`	`}`
`108`	`111`	`}`
Original file line number	Diff line number	Diff line change
`@@ -166,6 +166,10 @@ void TaskManager::EnterSleep(uint32_t microSeconds,`
`166`	`166`	`{`
`167`	`167`	`system_rtc_mem_write(RTC_MEM_SLEEP_ADDR, state, sizeofState);`
`168`	`168`	`}`
	`169`	`+ if (microSeconds == 0)`
	`170`	`+ {`
	`171`	`+ microSeconds = ESP.deepSleepMax();`
	`172`	`+ }`
`169`	`173`	`ESP.deepSleep(microSeconds, mode);`
`170`	`174`	`}`
`171`	`175`