Report useful error message on broken file(1).

[mmh] / uip / mhpath.c

diff --git a/uip/mhpath.c b/uip/mhpath.c
index 73d5e13..a8de5cb 100644 (file)
--- a/uip/mhpath.c
+++ b/uip/mhpath.c
@@ -11,7 +11,7 @@
 
 static struct swit switches[] = {
 #define VERSIONSW 0
-       { "version", 0 },
+       { "Version", 0 },
 #define HELPSW  1
        { "help", 0 },
        { NULL, 0 }
@@ -27,9 +27,7 @@ main(int argc, char **argv)
        struct msgs_array msgs = { 0, 0, NULL };
        struct msgs *mp;
 
-#ifdef LOCALE
        setlocale(LC_ALL, "");
-#endif
        invo_name = mhbasename(argv[0]);
 
        /* read user profile/context */
@@ -87,7 +85,7 @@ main(int argc, char **argv)
 
        /*
        ** We need to make sure there is message status space
-       ** for all the message numbers from 1 to "new" since
+       ** for all the message numbers from 1 to one beyond last since
        ** mhpath can select empty slots.  If we are adding
        ** space at the end, we go ahead and add 10 slots.
        */
@@ -98,8 +96,16 @@ main(int argc, char **argv)
                if (!(mp = folder_realloc(mp, 1, mp->hghoff)))
                        adios(NULL, "unable to allocate folder storage");
        }
+       /*
+       ** TODO: As folder_realloc() checks itself if the realloc
+       ** really is necesary, why don't we then:
+       **    if (!(mp = folder_realloc (mp, 1, mp->hghmsg+1)))
+       **        adios (NULL, "unable to allocate folder storage");
+       ** ? This at least appears most clear to me. -- meillo
+       */
+
 
-       mp->msgflags |= ALLOW_NEW;  /* allow the "new" sequence */
+       mp->msgflags |= ALLOW_BEYOND;  /* allow the beyond sequence */
 
        /* parse all the message ranges/sequences and set SELECTED */
        for (i = 0; i < msgs.size; i++)