From: markus schnalke Date: Tue, 24 Jan 2012 15:28:36 +0000 (+0100) Subject: Added a comment and a variable initialization. X-Git-Tag: mmh-thesis-end~416 X-Git-Url: http://git.marmaro.de/?p=mmh;a=commitdiff_plain;h=6dc010d334b477ad4dee0b1a1d3ee6a6bdb7bd9a Added a comment and a variable initialization. --- 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;