diff options
| author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2015-07-06 20:04:13 -0400 |
|---|---|---|
| committer | Sven Eden <yamakuzure@gmx.net> | 2017-03-14 10:05:57 +0100 |
| commit | c04b6b9bfe4a498a61ac34038bb9523ca9edfc36 (patch) | |
| tree | f5041ffd4efaba1ad5e0b755fbface8e2a7a027f /src/login/logind-seat.c | |
| parent | e2f0672c84ecd7c8a7507d6c1d7ebec1f6348524 (diff) | |
login: use normal comparison to zero for integers
! is supposed to be used for booleans and pointers.
Diffstat (limited to 'src/login/logind-seat.c')
| -rw-r--r-- | src/login/logind-seat.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/login/logind-seat.c b/src/login/logind-seat.c index 8198011c2..887de47f3 100644 --- a/src/login/logind-seat.c +++ b/src/login/logind-seat.c @@ -241,7 +241,7 @@ int seat_set_active(Seat *s, Session *session) { int seat_switch_to(Seat *s, unsigned int num) { /* Public session positions skip 0 (there is only F1-F12). Maybe it * will get reassigned in the future, so return error for now. */ - if (!num) + if (num == 0) return -EINVAL; if (num >= s->position_count || !s->positions[num]) { @@ -258,7 +258,7 @@ int seat_switch_to(Seat *s, unsigned int num) { int seat_switch_to_next(Seat *s) { unsigned int start, i; - if (!s->position_count) + if (s->position_count == 0) return -EINVAL; start = 1; @@ -279,7 +279,7 @@ int seat_switch_to_next(Seat *s) { int seat_switch_to_previous(Seat *s) { unsigned int start, i; - if (!s->position_count) + if (s->position_count == 0) return -EINVAL; start = 1; @@ -444,14 +444,14 @@ void seat_evict_position(Seat *s, Session *session) { session->pos = 0; - if (!pos) + if (pos == 0) return; if (pos < s->position_count && s->positions[pos] == session) { s->positions[pos] = NULL; /* There might be another session claiming the same - * position (eg., during gdm->session transition), so lets look + * position (eg., during gdm->session transition), so let's look * for it and set it on the free slot. */ LIST_FOREACH(sessions_by_seat, iter, s->sessions) { if (iter->pos == pos) { |