This was introduced in e1770af812 (2012-02-03, swap: replace failure boolean by result enum).
It doesn't do any harm as it most likely will get optimized away anyway, but maybe for readability? --- src/core/swap.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/core/swap.c b/src/core/swap.c index e0627db..6b204df 100644 --- a/src/core/swap.c +++ b/src/core/swap.c @@ -999,10 +999,7 @@ static void swap_sigchld_event(Unit *u, pid_t pid, int code, int status) { case SWAP_DEACTIVATING_SIGKILL: case SWAP_DEACTIVATING_SIGTERM: - if (f == SWAP_SUCCESS) - swap_enter_dead(s, f); - else - swap_enter_dead(s, f); + swap_enter_dead(s, f); break; default: -- 1.8.5.2.227.g53f3478 _______________________________________________ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel