Don't ever skip off duty, no matter if late or infinitely short.

Author: dok-net

cores/esp8266/core_esp8266_waveform.cpp

@@ -359,36 +359,21 @@ static ICACHE_RAM_ATTR void timer1Interrupt() {
             // active configuration and forward are 100% duty
             if (wave.periodCcys == wave.dutyCcys) {
               wave.nextPeriodCcy += periodCcys;
-              waveNextEventCcy = wave.endDutyCcy = wave.nextPeriodCcy;
+              wave.endDutyCcy = wave.nextPeriodCcy;
             }
             else {
               if (wave.autoPwm) {
-                if (static_cast<int32_t>(now - wave.nextPeriodCcy) >= 0) {
-                  wave.endDutyCcy += periodCcys - overshootCcys;
-                  wave.nextPeriodCcy += periodCcys;
-                  waveNextEventCcy = wave.endDutyCcy;
-                  // adapt expiry such that it occurs during intended cycle
-                  if (WaveformMode::EXPIRES == wave.mode)
-                    wave.expiryCcy += periodCcys;
-                }
-                else {
-                  wave.adjDutyCcys = overshootCcys;
-                  waveNextEventCcy = wave.nextPeriodCcy;
-                }
+                wave.adjDutyCcys += overshootCcys;
               }
-              else {
-                waveNextEventCcy = wave.nextPeriodCcy;
+              waveform.states ^= pinBit;
+              if (16 == pin) {
+                GP16O = 0;
               }
-              if (waveNextEventCcy == wave.nextPeriodCcy) {
-                waveform.states ^= pinBit;
-                if (16 == pin) {
-                  GP16O = 0;
-                }
-                else {
-                  GPOC = pinBit;
-                }
+              else {
+                GPOC = pinBit;
               }
             }
+            waveNextEventCcy = wave.nextPeriodCcy;
           }
           else {
             if (!wave.dutyCcys) {
@@ -400,8 +385,12 @@ static ICACHE_RAM_ATTR void timer1Interrupt() {
               int32_t dutyCcys = scaleCcys(wave.dutyCcys);
               if (dutyCcys > wave.adjDutyCcys) {
                 dutyCcys -= wave.adjDutyCcys;
+                wave.adjDutyCcys = 0;
+              }
+              else {
+                wave.adjDutyCcys -= dutyCcys;
+                dutyCcys = 0;
               }
-              wave.adjDutyCcys = 0;
               wave.endDutyCcy = now + dutyCcys;
               if (static_cast<int32_t>(wave.endDutyCcy - wave.nextPeriodCcy) >= 0) {
                 wave.endDutyCcy = wave.nextPeriodCcy;