From 3e3e0f988322e6cec76a78a4cdf68bd158d552e6 Mon Sep 17 00:00:00 2001 From: Ralph Corderoy Date: Mon, 17 Apr 2017 11:13:36 +0100 Subject: [PATCH] Don't store getname()'s return value during flushing. `cp' after the loop is overwritten before being read. Not storing the return value makes it clear it's not used. --- sbr/addrsbr.c | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/sbr/addrsbr.c b/sbr/addrsbr.c index 86b18a14..5ad8769e 100644 --- a/sbr/addrsbr.c +++ b/sbr/addrsbr.c @@ -310,12 +310,9 @@ ismymbox (struct mailname *np) return 0; } - /* - * Sigh, it turns out that the address parser gets messed up - * if you don't call getname() until it returns NULL. - */ - - while ((cp = getname(am)) != NULL) + /* Sigh, it turns out that the address parser gets messed up + * if you don't call getname() until it returns NULL. */ + while (getname(am) != NULL) ; } -- 2.56.0