Fix alarm cancel task triggered by same alarm

GeoTecINIT / nativescript-task-dispatcher

NativeScript-based mobile background task scheduler and dispatcher

Apache License 2.0

18 stars 6 forks source link

Fix alarm cancel task triggered by same alarm #15

Closed agonper closed 3 years ago

agonper commented 3 years ago

This PR fixes the following issue:

Sometimes, two alarms are running with different intervals which co-occur once every x runs. When one of the two tasks needed to be cancelled just before the next co-ocurring execution window the alarm was rescheduled, when it didn't need too, thus delaying other task's execution.

@matey97, I touched critical code parts while solving this issue. I would greatly appreciate if you can think of any scenarios where this code logic would fail. Thanks

agonper commented 3 years ago

I've just added new changes to try to reduce time drift between alarm executions to a minimum