devs/libstrophe-gh
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

handler: don't call time_stamp() twice in handler_fire_timed()

Browse Source
This commit is contained in:
Dmitry Podgorny
2017-06-25 21:07:48 +03:00
parent 82040403bf
commit e151096834
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/handler.c
View File

@@ -150,6 +150,7 @@ uint64_t handler_fire_timed(xmpp_ctx_t * const ctx)
xmpp_handlist_t *item, *next; xmpp_handlist_t *item, *next;
xmpp_conn_t *conn; xmpp_conn_t *conn;
uint64_t elapsed, min; uint64_t elapsed, min;
uint64_t timestamp;
int ret; int ret;
min = (uint64_t)(-1); min = (uint64_t)(-1);
@@ -176,10 +177,11 @@ uint64_t handler_fire_timed(xmpp_ctx_t * const ctx)
} }
next = item->next; next = item->next;
elapsed = time_elapsed(item->last_stamp, time_stamp()); timestamp = time_stamp();
elapsed = time_elapsed(item->last_stamp, timestamp);
if (elapsed >= item->period) { if (elapsed >= item->period) {
/* fire! */ /* fire! */
item->last_stamp = time_stamp(); item->last_stamp = timestamp;
ret = ((xmpp_timed_handler)item->handler)(conn, item->userdata); ret = ((xmpp_timed_handler)item->handler)(conn, item->userdata);
/* list may be changed during execution of a handler */ /* list may be changed during execution of a handler */
next = item->next; next = item->next;