From 6dc010d334b477ad4dee0b1a1d3ee6a6bdb7bd9a Mon Sep 17 00:00:00 2001 From: markus schnalke Date: Tue, 24 Jan 2012 16:28:36 +0100 Subject: [PATCH] Added a comment and a variable initialization. --- uip/mhpath.c | 8 ++++++++ uip/new.c | 2 +- 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/uip/mhpath.c b/uip/mhpath.c index 69ecaf2..5c0de05 100644 --- 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 2ac56d6..a4da635 100644 --- 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; -- 1.7.10.4