* patch #3968: Move the add() function from its own file (add.c) and

  into utils.c. There was also a duplicate add() function in mf.c which
  has been removed.

diff --git a/ChangeLog b/ChangeLog
index 1c2fc9ddc0b32fc725f204ad857b0f0afff9e84c..46d2bdbfafa352700f1fad42cc3b31ca01dcae0a 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2006-01-06  Josh Bressers <josh@bress.net>
+
+       * patch #3968: Move the add() function from its own file (add.c) and
+       into utils.c. There was also a duplicate add() function in mf.c which
+       has been removed.
+
 2006-01-02  Josh Bressers <josh@bress.net>
 
        * Remove sbr/pwd.c file, moving the pwd() function into sbr/utils.c.

diff --git a/h/prototypes.h b/h/prototypes.h
index fe41962a4da241114cdc31ded9fec16776cabdb9..d53d663027c5fde25435c799461e269f2839ec5e 100644 (file)
--- a/h/prototypes.h
+++ b/h/prototypes.h
@@ -25,7 +25,6 @@ char *etcpath(char *);
 /*
  * prototypes from the nmh subroutine library
  */
-char *add (char *, char *);
 void adios (char *, char *, ...);
 void admonish (char *, char *, ...);
 void advertise (char *, char *, char *, va_list);

diff --git a/h/utils.h b/h/utils.h
index 60e26e5d3909bacaeda0dfe807e5b0784ac20775..542baecafa3acbf6f171e425f9144c8ea151b1e3 100644 (file)
--- a/h/utils.h
+++ b/h/utils.h
@@ -8,3 +8,4 @@
 void *mh_xmalloc(size_t);
 void *mh_xrealloc(void *, size_t);
 char *pwd(void);
+char *add(char *, char *);

diff --git a/sbr/Makefile.in b/sbr/Makefile.in
index 5f73e0f4970f0fb4de14d851909ed6d1af430857..d6c2796247c830464d79c5a5fa38c74199a1c0da 100644 (file)
--- a/sbr/Makefile.in
+++ b/sbr/Makefile.in
@@ -52,7 +52,7 @@ COMPILE2 = $(CC) -c $(DEFS) $(CONFIGDEFS) $(INCLUDES) $(KRB4_INCLUDES) $(HESIOD_
 SIGNAL_H = @SIGNAL_H@
 
 # source for library functions
-SRCS = add.c addrsbr.c ambigsw.c atooi.c brkstring.c                   \
+SRCS = addrsbr.c ambigsw.c atooi.c brkstring.c                 \
        check_charset.c client.c closefds.c concat.c context_del.c      \
        context_find.c context_foil.c context_read.c                    \
        context_replace.c context_save.c copy.c                         \

diff --git a/sbr/add.c b/sbr/add.c
deleted file mode 100644 (file)
index 40201c1..0000000
--- a/sbr/add.c
+++ /dev/null
@@ -1,48 +0,0 @@
-
-/*
- * add.c -- If "s1" is NULL, this routine just creates a
- *       -- copy of "s2" into newly malloc'ed memory.
- *       --
- *       -- If "s1" is not NULL, then copy the concatenation
- *       -- of "s1" and "s2" (note the order) into newly
- *       -- malloc'ed memory.  Then free "s1".
- *
- * $Id$
- *
- * This code is Copyright (c) 2002, by the authors of nmh.  See the
- * COPYRIGHT file in the root directory of the nmh distribution for
- * complete copyright information.
- */
-
-#include <h/mh.h>
-#include <h/utils.h>
-
-char *
-add (char *s2, char *s1)
-{
-    char *cp;
-    size_t len1 = 0, len2 = 0;
-
-    if (s1)
-       len1 = strlen (s1);
-    if (s2)
-       len2 = strlen (s2);
-
-
-    cp = mh_xmalloc (len1 + len2 + 1);
-
-    /* Copy s1 and free it */
-    if (s1) {
-       memcpy (cp, s1, len1);
-       free (s1);
-    }
-
-    /* Copy s2 */
-    if (s2)
-       memcpy (cp + len1, s2, len2);
-
-    /* Now NULL terminate the string */
-    cp[len1 + len2] = '\0';
-
-    return cp;
-}

diff --git a/sbr/mf.c b/sbr/mf.c
index 518a184dd37f40f7d8d8e1c36fb635311a3ffc6d..44dab8a26b90f34d5ffa0dc08534755ee7e5f1fc 100644 (file)
--- a/sbr/mf.c
+++ b/sbr/mf.c
@@ -18,7 +18,6 @@
  * static prototypes
  */
 static char *getcpy (char *);
-static char *add (char *, char *);
 static void compress (char *, char *);
 static int isat (char *);
 static int parse_address (void);
@@ -49,20 +48,6 @@ getcpy (char *s)
 }
 
 
-static char *
-add (char *s1, char *s2)
-{
-    register char *p;
-
-    if (!s2)
-       return getcpy (s1);
-
-    p = mh_xmalloc ((size_t) (strlen (s1) + strlen (s2) + 2));
-    sprintf (p, "%s%s", s2, s1);
-    free (s2);
-    return p;
-}
-
 int
 isfrom(char *string)
 {

diff --git a/sbr/seq_read.c b/sbr/seq_read.c
index f36edb56cfb8cf5cc9f79d0b0d45088fd9dced2e..c18d1dae8a0525ab4ec9c333bb2ab0963d68518f 100644 (file)
--- a/sbr/seq_read.c
+++ b/sbr/seq_read.c
@@ -11,6 +11,7 @@
  */
 
 #include <h/mh.h>
+#include <h/utils.h>
 
 /*
  * static prototypes

diff --git a/sbr/utils.c b/sbr/utils.c
index 2120b84541cb77ee7d3933c00e07f6823ec90d5d..8741b2086f7a023f01b1910d3ff451625b1bed2d 100644 (file)
--- a/sbr/utils.c
+++ b/sbr/utils.c
@@ -74,3 +74,40 @@ pwd(void)
 
     return curwd;
 }
+
+/*
+ * add   -- If "s1" is NULL, this routine just creates a
+ *       -- copy of "s2" into newly malloc'ed memory.
+ *       --
+ *       -- If "s1" is not NULL, then copy the concatenation
+ *       -- of "s1" and "s2" (note the order) into newly
+ *       -- malloc'ed memory.  Then free "s1".
+ */
+char *
+add (char *s2, char *s1)
+{
+    char *cp;
+    size_t len1 = 0, len2 = 0;
+
+    if (s1)
+        len1 = strlen (s1);
+    if (s2)
+        len2 = strlen (s2);
+
+    cp = mh_xmalloc (len1 + len2 + 1);
+
+    /* Copy s1 and free it */
+    if (s1) {
+        memcpy (cp, s1, len1);
+        free (s1);
+    }
+
+    /* Copy s2 */
+    if (s2)
+        memcpy (cp + len1, s2, len2);
+
+    /* Now NULL terminate the string */
+    cp[len1 + len2] = '\0';
+
+    return cp;
+}

diff --git a/uip/ali.c b/uip/ali.c
index c6003879a8ef68c9772f04147c81b2a2171c6ae9..737f68c76b73239b2caf70759999958777d1f5b8 100644 (file)
--- a/uip/ali.c
+++ b/uip/ali.c
@@ -13,6 +13,7 @@
 #include <h/addrsbr.h>
 #include <h/aliasbr.h>
 #include <h/mts.h>
+#include <h/utils.h>
 
 /*
  * maximum number of names

diff --git a/uip/distsbr.c b/uip/distsbr.c
index bda98c8f120b9266c6ddad80c6075673bc5c142c..0191037691f388840a6b86650b4ee86c3cb198aa 100644 (file)
--- a/uip/distsbr.c
+++ b/uip/distsbr.c
@@ -11,6 +11,7 @@
 
 #include <h/mh.h>
 #include <fcntl.h>
+#include <h/utils.h>
 
 static int  hdrfd = NOTOK;
 static int  txtfd = NOTOK;

diff --git a/uip/mhbuild.c b/uip/mhbuild.c
index b16e8c96b4948c7d56ae109d93f83a6885bbfc48..1c5abdc3c42c3211d7b039be7009a12612ae28fe 100644 (file)
--- a/uip/mhbuild.c
+++ b/uip/mhbuild.c
@@ -20,6 +20,7 @@
 #include <h/mime.h>
 #include <h/mhparse.h>
 #include <h/mhcachesbr.h>
+#include <h/utils.h>
 
 #ifdef HAVE_SYS_WAIT_H
 # include <sys/wait.h>

diff --git a/uip/mhcachesbr.c b/uip/mhcachesbr.c
index 56d3dcc4d8be120f6ff4715f5f66a3e09d6ee583..ba66f6ebf5126a76f9fa86fde6cce63a80681e24 100644 (file)
--- a/uip/mhcachesbr.c
+++ b/uip/mhcachesbr.c
@@ -21,6 +21,7 @@
 #include <h/mime.h>
 #include <h/mhparse.h>
 #include <h/mhcachesbr.h>
+#include <h/utils.h>
 
 #ifdef TIME_WITH_SYS_TIME
 # include <sys/time.h>

diff --git a/uip/mhlistsbr.c b/uip/mhlistsbr.c
index bce44d0726e6fe7ebd46496896c3f6b7ca77fff1..10ec55d637b9a973663810be342b3d28bcc0fbff 100644 (file)
--- a/uip/mhlistsbr.c
+++ b/uip/mhlistsbr.c
@@ -19,6 +19,7 @@
 #include <h/tws.h>
 #include <h/mime.h>
 #include <h/mhparse.h>
+#include <h/utils.h>
 
 /* mhmisc.c */
 int part_ok (CT, int);

diff --git a/uip/mhmail.c b/uip/mhmail.c
index d74a7f1d4b4f8b1302912e8c318454d310c5322e..b5da86c6f583301e27091cd7afda8893ca9d0ee6 100644 (file)
--- a/uip/mhmail.c
+++ b/uip/mhmail.c
@@ -11,6 +11,7 @@
 
 #include <h/mh.h>
 #include <h/signals.h>
+#include <h/utils.h>
 #include <signal.h>
 
 static struct swit switches[] = {

diff --git a/uip/mhmisc.c b/uip/mhmisc.c
index a3da759fcd7804408402bf2c82dcff6fe248c71c..9f0044ac15025fd59a08e4f71425e9e1d9dd4760 100644 (file)
--- a/uip/mhmisc.c
+++ b/uip/mhmisc.c
@@ -13,6 +13,7 @@
 #include <errno.h>
 #include <h/mime.h>
 #include <h/mhparse.h>
+#include <h/utils.h>
 
 extern int debugsw;
 

diff --git a/uip/mhshowsbr.c b/uip/mhshowsbr.c
index 1c786cc664b8e5d0c5fc7bc9f72f3142fe651751..08d676e035626d7f2046c2cb2ac62f681ebc1e55 100644 (file)
--- a/uip/mhshowsbr.c
+++ b/uip/mhshowsbr.c
@@ -20,6 +20,7 @@
 #include <h/tws.h>
 #include <h/mime.h>
 #include <h/mhparse.h>
+#include <h/utils.h>
 
 #ifdef HAVE_SYS_WAIT_H
 # include <sys/wait.h>

diff --git a/uip/mhstoresbr.c b/uip/mhstoresbr.c
index 72694b673ef0b40468249748af13fbd7c353e265..519e4810e1eb808c851e552dae0fa4f42f4f8f23 100644 (file)
--- a/uip/mhstoresbr.c
+++ b/uip/mhstoresbr.c
@@ -20,6 +20,7 @@
 #include <h/tws.h>
 #include <h/mime.h>
 #include <h/mhparse.h>
+#include <h/utils.h>
 
 
 /*

diff --git a/uip/picksbr.c b/uip/picksbr.c
index 71ae644feb76e86ce3de36196e627316ec9fea8f..1154f46c50b947a6a1b2d81d635cc253927df312 100644 (file)
--- a/uip/picksbr.c
+++ b/uip/picksbr.c
@@ -12,6 +12,7 @@
 #include <h/mh.h>
 #include <h/tws.h>
 #include <h/picksbr.h>
+#include <h/utils.h>
 
 #ifdef TIME_WITH_SYS_TIME
 # include <sys/time.h>

diff --git a/uip/post.c b/uip/post.c
index aff49a09197e0d387ec63878e2889cf1123f7fd2..2fe45a1081c86729d9d0fbded1144d49f697a67b 100644 (file)
--- a/uip/post.c
+++ b/uip/post.c
@@ -16,6 +16,7 @@
 #include <h/aliasbr.h>
 #include <h/dropsbr.h>
 #include <h/mime.h>
+#include <h/utils.h>
 
 #include <h/tws.h>
 #include <h/mts.h>

diff --git a/uip/repl.c b/uip/repl.c
index b1c0782f0381ca8a4be2ce8242f1f30ebfcb8536..e5fa527d6aee378764015a66f6c8aa74afb0a885 100644 (file)
--- a/uip/repl.c
+++ b/uip/repl.c
@@ -10,6 +10,7 @@
  */
 
 #include <h/mh.h>
+#include <h/utils.h>
 
 
 static struct swit switches[] = {

diff --git a/uip/show.c b/uip/show.c
index 5773b56ea4772bcfe9748d53f67eafb7ff3c00dd..b068d4952be836ec1f98423b4453b43bd9f96bc9 100644 (file)
--- a/uip/show.c
+++ b/uip/show.c
@@ -11,6 +11,7 @@
 
 #include <h/mh.h>
 #include <h/mime.h>
+#include <h/utils.h>
 
 static struct swit switches[] = {
 #define CHECKMIMESW          0

diff --git a/uip/slocal.c b/uip/slocal.c
index 993a141a0c3a6f217598554f9aad15bf88ee931c..e51f869c0cf78b5d05ba407a9d9c52691d6e2691 100644 (file)
--- a/uip/slocal.c
+++ b/uip/slocal.c
@@ -32,6 +32,7 @@
 #include <h/signals.h>
 #include <h/tws.h>
 #include <h/mts.h>
+#include <h/utils.h>
 
 #include <pwd.h>
 #include <signal.h>

diff --git a/uip/spost.c b/uip/spost.c
index 5a7d9435a51194c156415eb2c54990cf61901483..e2b43bd6350b4d68db03e335c9f9417539e3e7f8 100644 (file)
--- a/uip/spost.c
+++ b/uip/spost.c
@@ -19,6 +19,7 @@
 #include <h/dropsbr.h>
 #include <h/tws.h>
 #include <h/mts.h>
+#include <h/utils.h>
 
 #define        uptolow(c)      ((isalpha(c) && isupper (c)) ? tolower (c) : c)