Skip to content

Commit

Permalink
[alarmtracker] Refactor alarm state tracking functionality
Browse files Browse the repository at this point in the history
The code is hard to read, potentially leaves active timers behind
on unload, alarm state exposed to libdsme clients differs from
one used within dsme, etc.

Group and name functions by type, improve debug logging, add
doxygen comments.

There should be no functional changes, except: re-evaluate timer
is cancelled on unload, and alarm state notifications within
dsme and via libdsme are unified.

Signed-off-by: Simo Piiroinen <simo.piiroinen@jollamobile.com>
  • Loading branch information
spiiroin committed May 16, 2019
1 parent 894d98a commit a1e65a4
Show file tree
Hide file tree
Showing 2 changed files with 434 additions and 207 deletions.
2 changes: 1 addition & 1 deletion Makefile.custom
Expand Up @@ -328,6 +328,7 @@ CLEAN_SOURCES += dsme/dsmesock.c
CLEAN_SOURCES += dsme/timers.c
CLEAN_SOURCES += dsme/utility.c
CLEAN_SOURCES += modules/abootsettings.c
CLEAN_SOURCES += modules/alarmtracker.c
CLEAN_SOURCES += modules/batterytracker.c
CLEAN_SOURCES += modules/dbusautoconnector.c
CLEAN_SOURCES += modules/dbusproxy.c
Expand Down Expand Up @@ -374,7 +375,6 @@ DIRTY_SOURCES += dsme/mainloop.c
DIRTY_SOURCES += dsme/modulebase.c
DIRTY_SOURCES += dsme/oom.c
DIRTY_SOURCES += getbootstate/getbootstate.c
DIRTY_SOURCES += modules/alarmtracker.c
DIRTY_SOURCES += modules/bootreasonlogger.c
DIRTY_SOURCES += modules/emergencycalltracker.c
DIRTY_SOURCES += modules/iphb.c
Expand Down

0 comments on commit a1e65a4

Please sign in to comment.