* movemail.c: Define macros only in needed contexts.

diff --git a/lib-src/ChangeLog b/lib-src/ChangeLog
index 1d0f24212adef86263b1a6e7d17e8b7fa8af5b04..7aa13a45cbad710e18a6412728c0ff1d2dc059d3 100644 (file)
--- a/lib-src/ChangeLog
+++ b/lib-src/ChangeLog
@@ -14,8 +14,9 @@
 
        * movemail.c (main, pop_retr): Rename locals to avoid shadowing.
        (progname, sfi, sfo, ibuffer, obuffer): Remove unused vars.
-       (DIRECTORY_SEP, IS_DIRECTORY_SEP, DONE, IS_FROM_LINE):
-       Remove unused macros.
+       (DONE): Remove unused macro.
+       (DIRECTORY_SEP, IS_DIRECTORY_SEP, IS_FROM_LINE):
+       Define these macros only in the contexts that need them.
        * pop.c (index): Remove unused macro.
        (KPOP_PORT): Define only if KERBEROS is defined.
 

diff --git a/lib-src/movemail.c b/lib-src/movemail.c
index bc7fa8824e294775b719c51162c01cdb7d99295e..4a894c1cba1a450648b153b0d4e4a6632a095449 100644 (file)
--- a/lib-src/movemail.c
+++ b/lib-src/movemail.c
@@ -262,6 +262,13 @@ main (int argc, char **argv)
   if (! spool_name)
 #endif
     {
+      #ifndef DIRECTORY_SEP
+       #define DIRECTORY_SEP '/'
+      #endif
+      #ifndef IS_DIRECTORY_SEP
+       #define IS_DIRECTORY_SEP(_c_) ((_c_) == DIRECTORY_SEP)
+      #endif
+
       /* Use a lock file named after our first argument with .lock appended:
         If it exists, the mail file is locked.  */
       /* Note: this locking mechanism is *required* by the mailer
@@ -862,6 +869,12 @@ static int
 mbx_write (char *line, int len, FILE *mbf)
 {
 #ifdef MOVEMAIL_QUOTE_POP_FROM_LINES
+  /* Do this as a macro instead of using strcmp to save on execution time. */
+  # define IS_FROM_LINE(a) ((a[0] == 'F')      \
+                           && (a[1] == 'r')    \
+                           && (a[2] == 'o')    \
+                           && (a[3] == 'm')    \
+                           && (a[4] == ' '))
   if (IS_FROM_LINE (line))
     {
       if (fputc ('>', mbf) == EOF)