Previously contained the code:
authorDan Harkless <dan@harkless.org>
Fri, 16 Jul 1999 01:16:08 +0000 (01:16 +0000)
committerDan Harkless <dan@harkless.org>
Fri, 16 Jul 1999 01:16:08 +0000 (01:16 +0000)
if (numburst == 0)
    if (!quietsw)
admonish (NULL, "message %d not in digest format",
  msgnum);
else
    adios (NULL, "burst() botch -- you lose big");

If the indentation is to be believed, this is a bug.  I trusted the indentation
and added {}s around the inner if().

uip/burst.c

index d13b076..d367b97 100644 (file)
@@ -154,9 +154,11 @@ main (int argc, char **argv)
                            numburst, numburst > 1 ? "s" : "", msgnum);
                burst (&mp, msgnum, smsgs, numburst, inplace, verbosw);
            } else {
-               if (numburst == 0)
+               if (numburst == 0) {
                    if (!quietsw)
-                       admonish (NULL, "message %d not in digest format", msgnum);
+                       admonish (NULL, "message %d not in digest format",
+                                 msgnum);
+               }  /* this pair of braces was missing before 1999-07-15 */
                else
                    adios (NULL, "burst() botch -- you lose big");
            }
@@ -183,7 +185,7 @@ main (int argc, char **argv)
     seq_save (mp);     /* synchronize message sequences */
     context_save ();   /* save the context file         */
     folder_free (mp);  /* free folder/message structure */
-    done (0);
+    return done (0);
 }