Remove alwaysExpired, it is identical to 0 and obfuscates the code.

Author: dok-net

cores/esp8266/PolledTimeout.h

@@ -146,7 +146,6 @@ class timeoutTemplate
   using timeType = typename TimePolicyT::timeType;
   static_assert(std::is_unsigned<timeType>::value == true, "timeType must be unsigned");
 
-  static constexpr timeType alwaysExpired   = 0;
   static constexpr timeType neverExpires    = std::numeric_limits<timeType>::max();
   static constexpr timeType rangeCompensate = TimePolicyT::rangeCompensate; //debug
 
@@ -177,7 +176,7 @@ class timeoutTemplate
 
   bool canWait () const
   {
-    return _timeout != alwaysExpired && !_oneShotExpired;
+    return _timeout != 0 && !_oneShotExpired;
   }
 
   // Resets, will trigger after this new timeout.
@@ -216,7 +215,7 @@ class timeoutTemplate
 
   void resetToNeverExpires ()
   {
-    _timeout = alwaysExpired + 1; // because canWait() has precedence
+    _timeout = 1; // because canWait() has precedence
     _neverExpires = true;
   }
 

cores/esp8266/Stream.h

@@ -171,7 +171,7 @@ class Stream: public Print {
 
         // transfers already buffered / immediately available data (no timeout)
         // returns number of transferred bytes
-        [[deprecated]] size_t sendAvailable (Print* to) { return sendGeneric(to, -1, -1, oneShotMs::alwaysExpired); }
+        [[deprecated]] size_t sendAvailable (Print* to) { return sendGeneric(to, -1, -1, 0); }
         [[deprecated]] size_t sendAvailable (Print& to) { return sendAvailable(&to); }
 
         // transfers data until timeout