diff options
| author | Lennart Poettering <lennart@poettering.net> | 2015-01-28 15:07:13 +0100 |
|---|---|---|
| committer | Lennart Poettering <lennart@poettering.net> | 2015-01-28 15:07:13 +0100 |
| commit | 82a2b6bb5e4e5d294f09af778c48974a7857afb6 (patch) | |
| tree | 37d7926f1c1e575d8e1344890953ca6a0114a011 /src/core/timer.c | |
| parent | 3ad85e3775d018f32098a2957ce25e1f53440da1 (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.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/core/timer.c b/src/core/timer.c index 309852aae..45744c7de 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) { |