expiredOneShot() qualifed as const, despite asymmetry with expired() and expiredRetrigger().

dok-net · dok-net · commit 496d5579d4ba · 2023-07-01T12:07:40.000+02:00
_oneShotExpired qualified as mutable member.
This is done as intermediate fix for use in derived class ESP8266WiFiMesh ExpiringTimeTracker.
diff --git a/cores/esp8266/PolledTimeout.h b/cores/esp8266/PolledTimeout.h
@@ -270,7 +270,7 @@ class timeoutTemplate
   }
 
   IRAM_ATTR // fast
-  bool expiredOneShot()
+  bool expiredOneShot() const
   {
     // returns "always expired" or "has expired"
     if (!canWait()) return true;
@@ -285,7 +285,7 @@ class timeoutTemplate
   timeType _timeout;
   timeType _start;
   bool _neverExpires;
-  bool _oneShotExpired;
+  mutable bool _oneShotExpired;
 };
 
 // legacy type names, deprecated (unit is milliseconds)
diff --git a/libraries/ESP8266WiFiMesh/src/ExpiringTimeTracker.h b/libraries/ESP8266WiFiMesh/src/ExpiringTimeTracker.h
@@ -61,7 +61,7 @@ class ExpiringTimeTracker : private esp8266::polledTimeout::oneShotMs {
    * Get the time since the ExpiringTimeTracker instance creation or the last reset(), whichever is more recent.
    */
   uint32_t elapsedTime() const;
-  bool expired();
+  bool expired() const;
   void reset();
   void reset(const uint32_t newDuration);
   void reset(const calculatorType newDurationCalculator);
