Tighten screen sorting: comparing a screen with itself should return 0

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;
 }