Added a comment and a variable initialization.

diff --git a/uip/mhpath.c b/uip/mhpath.c
index 69ecaf2db4d21e72307520d03d90badff3f958f4..5c0de0577d7c4c9e97cb508000109c14adf5bd01 100644 (file)
--- a/uip/mhpath.c
+++ b/uip/mhpath.c
@@ -98,6 +98,14 @@ 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_BEYOND;  /* allow the beyond sequence */
 

diff --git a/uip/new.c b/uip/new.c
index 2ac56d63c648608e37a270867068a4a876be03fc..a4da635792bf3f0a887d530fa5ccfd85211d1106 100644 (file)
--- a/uip/new.c
+++ b/uip/new.c
@@ -308,7 +308,7 @@ join_sequences(char *sequences[])
 static struct node *
 doit(char *cur, char *folders, char *sequences[])
 {
-       struct node *first, *cur_node, *node, *last, *prev;
+       struct node *first, *cur_node, *node, *last = NULL, *prev;
        size_t folder_len;
        int count, total = 0;
        char *command = NULL, *sequences_s = NULL;