From: Ralph Corderoy Date: Wed, 23 Aug 2017 20:25:52 +0000 (+0100) Subject: Remove the `else' after a simple if-then block ending in `break'. X-Git-Url: https://diplodocus.org/git/nmh/commitdiff_plain/ab2201f516946860e9322363be090a370da675fd Remove the `else' after a simple if-then block ending in `break'. Before After if (foo) if (foo) break; break; else bar(); bar(); --- diff --git a/sbr/vfgets.c b/sbr/vfgets.c index 36f71379..480af139 100644 --- a/sbr/vfgets.c +++ b/sbr/vfgets.c @@ -44,8 +44,7 @@ wrong_guess: for (fp = dp - 1, toggle = 0; fp >= cp; fp--) { if (*fp != QUOTE) break; - else - toggle = !toggle; + toggle = !toggle; } if (toggle) goto wrong_guess; diff --git a/uip/burst.c b/uip/burst.c index de762e99..e5020cfd 100644 --- a/uip/burst.c +++ b/uip/burst.c @@ -293,8 +293,7 @@ find_delim (int msgnum, struct smsg *smsgs, int *mimesw) for (c = 0; fgets (buffer, sizeof(buffer), in); c = buffer[0]) { if ((wasdlm = CHECKDELIM(buffer))) break; - else - pos += (long) strlen (buffer); + pos += (long) strlen (buffer); } /* diff --git a/uip/prompter.c b/uip/prompter.c index 5c4926a0..878b2b43 100644 --- a/uip/prompter.c +++ b/uip/prompter.c @@ -264,8 +264,7 @@ abort: state = m_getfld2(&gstate, name, field, &fieldsz))); if (prepend || !body) break; - else - printf ("\n--------Enter additional text\n\n"); + printf ("\n--------Enter additional text\n\n"); } fflush (stdout);