mirror/btstack
1
0
Fork 0
mirror of https://github.com/bluekitchen/btstack.git synced 2025-04-11 00:44:27 +00:00
Code Issues Packages Projects Releases Wiki Activity

btstack_run_loop: add log_error and explanation for assert on adding already active timer

Browse Source
This commit is contained in:
Matthias Ringwald 2022-09-14 12:07:40 +02:00
parent 4856283b35
commit 5332c7c0a9
1 changed files with 18 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
src/btstack_run_loop.c
View File

@ -94,7 +94,24 @@ void btstack_run_loop_base_add_timer(btstack_timer_source_t * timer){
btstack_linked_item_t *it;
for (it = (btstack_linked_item_t *) &btstack_run_loop_base_timers; it->next ; it = it->next){
btstack_timer_source_t * next = (btstack_timer_source_t *) it->next;
btstack_assert(next != timer);
if (next == timer){
log_error("Timer %p already registered! Please read source code comment.", timer);
//
// Dear BTstack User!
//
// If you hit the assert below, your application code tried to add a timer to the list of
// timers that's already in the timer list, i.e., it's already registered.
//
// As you've probably already modified the timer, just ignoring this might lead to unexpected
// and hard to debug issues. Instead, we decided to raise an assert in this case to help.
//
// Please do a backtrace and check where you register this timer.
// If you just want to restart it you can call btstack_run_loop_timer_remove(..) before restarting the timer.
//
btstack_assert(false);
}
int32_t delta = btstack_time_delta(timer->timeout, next->timeout);
if (delta < 0) break;
}