From 2e150215834904311e167edb9eefac4aee023fc5 Mon Sep 17 00:00:00 2001 From: Ralph Corderoy Date: Sat, 9 Sep 2017 18:28:31 +0100 Subject: [PATCH] folder_pack.c: Flip if-condition so return moves from else block. Then no need for else block after return. --- sbr/folder_pack.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/sbr/folder_pack.c b/sbr/folder_pack.c index 82a366c8..22c02079 100644 --- a/sbr/folder_pack.c +++ b/sbr/folder_pack.c @@ -33,12 +33,11 @@ folder_pack (struct msgs **mpp, int verbose) * for all numbers from 1 to current high message. */ if (mp->lowoff > 1) { - if ((mp = folder_realloc (mp, 1, mp->hghmsg))) - *mpp = mp; - else { + if (!(mp = folder_realloc(mp, 1, mp->hghmsg))) { inform("unable to allocate folder storage"); return -1; } + *mpp = mp; } for (msgnum = mp->lowmsg, hole = 1; msgnum <= mp->hghmsg; msgnum++) { -- 2.56.0