Added a comment and a variable initialization.

[mmh] / uip / mhpath.c

diff --git a/uip/mhpath.c b/uip/mhpath.c
index 69ecaf2..5c0de05 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 */