From 4aace27b5eeab890bdab8e1e1725b02ab435ff6a Mon Sep 17 00:00:00 2001 From: Ralph Corderoy Date: Thu, 20 Oct 2016 23:30:39 +0100 Subject: [PATCH] Don't `else' after return. Simplify control flow. --- uip/sortm.c | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/uip/sortm.c b/uip/sortm.c index fbedaf26..9351ec03 100644 --- a/uip/sortm.c +++ b/uip/sortm.c @@ -477,12 +477,11 @@ dsort (struct smsg **a, struct smsg **b) { if ((*a)->s_clock < (*b)->s_clock) return (-1); - else if ((*a)->s_clock > (*b)->s_clock) + if ((*a)->s_clock > (*b)->s_clock) return (1); - else if ((*a)->s_msg < (*b)->s_msg) + if ((*a)->s_msg < (*b)->s_msg) return (-1); - else - return (1); + return (1); } /* @@ -506,10 +505,9 @@ txtsort (struct smsg **a, struct smsg **b) if ((i = strcmp ((*a)->s_subj, (*b)->s_subj))) return (i); - else if ((*a)->s_msg < (*b)->s_msg) + if ((*a)->s_msg < (*b)->s_msg) return (-1); - else - return (1); + return (1); } static void -- 2.56.0