Merge pull request #19 from RushOnline/fix-reattach-interrupt

khoih-prog · web-flow · commit 7f86f9ca09a6 · 2022-01-18T15:58:09.000-05:00
bugfix: reattachInterrupt() pass wrong frequency value to setFrequency()
diff --git a/src/ESP8266TimerInterrupt.h b/src/ESP8266TimerInterrupt.h
@@ -180,7 +180,7 @@ class ESP8266TimerInterrupt
     void reattachInterrupt()
     {
       if ( (_frequency != 0) && (_timerCount != 0) && (_callback != NULL) )
-        setFrequency(_frequency, _callback);
+        setFrequency(_frequency / _timerCount, _callback);
     }
 
     // Duration (in milliseconds). Duration = 0 or not specified => run indefinitely
diff --git a/src_cpp/ESP8266TimerInterrupt.h b/src_cpp/ESP8266TimerInterrupt.h
@@ -180,7 +180,7 @@ class ESP8266TimerInterrupt
     void reattachInterrupt()
     {
       if ( (_frequency != 0) && (_timerCount != 0) && (_callback != NULL) )
-        setFrequency(_frequency, _callback);
+        setFrequency(_frequency / _timerCount, _callback);
     }
 
     // Duration (in milliseconds). Duration = 0 or not specified => run indefinitely
diff --git a/src_h/ESP8266TimerInterrupt.h b/src_h/ESP8266TimerInterrupt.h
@@ -180,7 +180,7 @@ class ESP8266TimerInterrupt
     void reattachInterrupt()
     {
       if ( (_frequency != 0) && (_timerCount != 0) && (_callback != NULL) )
-        setFrequency(_frequency, _callback);
+        setFrequency(_frequency / _timerCount, _callback);
     }
 
     // Duration (in milliseconds). Duration = 0 or not specified => run indefinitely

Original file line number	Diff line number	Diff line change
`@@ -180,7 +180,7 @@ class ESP8266TimerInterrupt`
`180`	`180`	`void reattachInterrupt()`
`181`	`181`	`{`
`182`	`182`	`if ( (_frequency != 0) && (_timerCount != 0) && (_callback != NULL) )`
`183`		`- setFrequency(_frequency, _callback);`
	`183`	`+ setFrequency(_frequency / _timerCount, _callback);`
`184`	`184`	`}`
`185`	`185`
`186`	`186`	`// Duration (in milliseconds). Duration = 0 or not specified => run indefinitely`