Previously contained the code:

		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().

diff --git a/uip/burst.c b/uip/burst.c
index d13b0760bcade1bb8fdafda2031c1a106396335f..d367b97dd779c0c64fdb3adb8fd1d3723ee2c796 100644 (file)
--- a/uip/burst.c
+++ b/uip/burst.c
@@ -154,9 +154,11 @@ main (int argc, char **argv)
                            numburst, numburst > 1 ? "s" : "", msgnum);
                burst (&mp, msgnum, smsgs, numburst, inplace, verbosw);
            } else {
                            numburst, numburst > 1 ? "s" : "", msgnum);
                burst (&mp, msgnum, smsgs, numburst, inplace, verbosw);
            } else {

-               if (numburst == 0)
+               if (numburst == 0) {

                    if (!quietsw)
                    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");
            }
                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 */
     seq_save (mp);     /* synchronize message sequences */
     context_save ();   /* save the context file         */
     folder_free (mp);  /* free folder/message structure */

-    done (0);
+    return done (0);

 }
 
 
 }