]> diplodocus.org Git - nmh/commitdiff
Don't store getname()'s return value during flushing.
authorRalph Corderoy <ralph@inputplus.co.uk>
Mon, 17 Apr 2017 10:13:36 +0000 (11:13 +0100)
committerRalph Corderoy <ralph@inputplus.co.uk>
Mon, 17 Apr 2017 10:13:36 +0000 (11:13 +0100)
`cp' after the loop is overwritten before being read.
Not storing the return value makes it clear it's not used.

sbr/addrsbr.c

index 86b18a1..5ad8769 100644 (file)
@@ -310,12 +310,9 @@ ismymbox (struct mailname *np)
                return 0;
            }
 
                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)
                ;
        }
 
                ;
        }