diff options
author | Jérémie Courrèges-Anglas <jca@wxcvbn.org> | 2016-12-01 22:12:29 +0100 |
---|---|---|
committer | Jérémie Courrèges-Anglas <jca@wxcvbn.org> | 2016-12-01 22:12:29 +0100 |
commit | 3d24fae3ad1c59f7baa6dd90dd53bda39dbddf06 (patch) | |
tree | 3ad360f92d584fcc531ada6df8a7aa8646977f4d | |
parent | 3a784b57107d6bf1d65e6b01f744e91c2ec790c6 (diff) | |
download | ratpoison-3d24fae3ad1c59f7baa6dd90dd53bda39dbddf06.zip |
Tighten screen sorting: comparing a screen with itself should return 0
-rw-r--r-- | src/screen.c | 22 |
1 files changed, 8 insertions, 14 deletions
diff --git a/src/screen.c b/src/screen.c index 7593f49..9c7f1e6 100644 --- a/src/screen.c +++ b/src/screen.c @@ -204,20 +204,14 @@ screen_cmp (void *priv, struct list_head *a, struct list_head *b) rp_screen *sc_b = container_of (b, typeof(*sc_b), node); if (sc_a->left < sc_b->left) - { - return -1; - } - else if (sc_a->left > sc_b->left) - { - return 1; - } - else if (sc_a->left == sc_b->left) - { - if (sc_a->top < sc_b->top) - return -1; - else - return 1; - } + return -1; + if (sc_a->left > sc_b->left) + return 1; + + if (sc_a->top > sc_b->top) + return -1; + if (sc_a->top < sc_b->top) + return 1; return 0; } |