summaryrefslogtreecommitdiff
path: root/src/core/timer.c
diff options
context:
space:
mode:
authorLennart Poettering <lennart@poettering.net>2015-01-28 15:07:13 +0100
committerLennart Poettering <lennart@poettering.net>2015-01-28 15:07:13 +0100
commit82a2b6bb5e4e5d294f09af778c48974a7857afb6 (patch)
tree37d7926f1c1e575d8e1344890953ca6a0114a011 /src/core/timer.c
parent3ad85e3775d018f32098a2957ce25e1f53440da1 (diff)
core: output unit status output strings to console, only if we actually are changing unit state
Unit _start() and _stop() implementations can fail with -EAGAIN to delay execution temporarily. Thus, we should not output status messages before invoking these calls, but after, and only when we know that the invocation actually made a change.
Diffstat (limited to 'src/core/timer.c')
-rw-r--r--src/core/timer.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/core/timer.c b/src/core/timer.c
index 309852aae6..45744c7de5 100644
--- a/src/core/timer.c
+++ b/src/core/timer.c
@@ -548,7 +548,7 @@ static int timer_start(Unit *u) {
t->result = TIMER_SUCCESS;
timer_enter_waiting(t, true);
- return 0;
+ return 1;
}
static int timer_stop(Unit *u) {
@@ -558,7 +558,7 @@ static int timer_stop(Unit *u) {
assert(t->state == TIMER_WAITING || t->state == TIMER_RUNNING || t->state == TIMER_ELAPSED);
timer_enter_dead(t, TIMER_SUCCESS);
- return 0;
+ return 1;
}
static int timer_serialize(Unit *u, FILE *f, FDSet *fds) {