From 629796cce723b508e636bcb48e89c79f61bc0285 Mon Sep 17 00:00:00 2001 From: Sami Kerola Date: Fri, 9 Nov 2012 07:55:48 +0000 Subject: [PATCH] sort: fix if else return statements If something is true, and it returns using else is pointless. Signed-off-by: Sami Kerola --- src/sort.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/src/sort.c b/src/sort.c index e956608..9939e61 100644 --- a/src/sort.c +++ b/src/sort.c @@ -48,10 +48,9 @@ int intcomp(const void *restrict x, const void *restrict y) { if (*(uint32_t *) x < *(uint32_t *) y) return -1; - else if (*(uint32_t *) y < *(uint32_t *) x) + if (*(uint32_t *) y < *(uint32_t *) x) return 1; - else - return 0; + return 0; } int rangecomp(const void *restrict r1, const void *restrict r2) @@ -59,11 +58,10 @@ int rangecomp(const void *restrict r1, const void *restrict r2) if ((((struct range_t *)r1)->first_ip) < (((struct range_t *)r2)->first_ip)) return -1; - else if ((((struct range_t *)r2)->first_ip) < + if ((((struct range_t *)r2)->first_ip) < (((struct range_t *)r1)->first_ip)) return 1; - else - return 0; + return 0; } unsigned long int ret_ip(struct range_t r)