# HG changeset patch # User Andreas Schwab # Date 1278417035 -7200 # Node ID 3f9a67cdf9f7e491a452ed48a809fa448628f31f # Parent 5833ef1423747ba943c526da283f5481a15b069d Clean up movemail source * movemail.c: Add MAIL_USE_POP around prototypes. Include if HAVE_STRING_H. (strerror): Only declare if !HAVE_STRERROR. (fatal): Make static. (error): Likewise. (pfatal_with_name): Likewise. (pfatal_and_delete). Likewise. (concat): Likewise. (xmalloc): Likewise. (popmail): Likewise. (pop_retr): Likewise. (mbx_write): Likewise. (mbx_delimit_begin): Likewise. (mbx_delimit_end): Likewise. diff -r 5833ef142374 -r 3f9a67cdf9f7 lib-src/ChangeLog --- a/lib-src/ChangeLog Tue Jul 06 13:11:28 2010 +0200 +++ b/lib-src/ChangeLog Tue Jul 06 13:50:35 2010 +0200 @@ -1,6 +1,19 @@ 2010-07-06 Andreas Schwab * movemail.c: Add MAIL_USE_POP around prototypes. + Include if HAVE_STRING_H. + (strerror): Only declare if !HAVE_STRERROR. + (fatal): Make static. + (error): Likewise. + (pfatal_with_name): Likewise. + (pfatal_and_delete). Likewise. + (concat): Likewise. + (xmalloc): Likewise. + (popmail): Likewise. + (pop_retr): Likewise. + (mbx_write): Likewise. + (mbx_delimit_begin): Likewise. + (mbx_delimit_end): Likewise. 2010-07-04 Dan Nicolaescu diff -r 5833ef142374 -r 3f9a67cdf9f7 lib-src/movemail.c --- a/lib-src/movemail.c Tue Jul 06 13:11:28 2010 +0200 +++ b/lib-src/movemail.c Tue Jul 06 13:50:35 2010 +0200 @@ -69,6 +69,9 @@ #ifdef HAVE_FCNTL_H #include #endif +#ifdef HAVE_STRING_H +#include +#endif #include "syswait.h" #ifdef MAIL_USE_POP #include "pop.h" @@ -140,7 +143,9 @@ #endif #endif +#ifndef HAVE_STRERROR char *strerror (int); +#endif #ifdef HAVE_INDEX extern char *index (const char *, int); #endif @@ -148,18 +153,18 @@ extern char *rindex (const char *, int); #endif -void fatal (char *s1, char *s2, char *s3); -void error (char *s1, char *s2, char *s3); -void pfatal_with_name (char *name); -void pfatal_and_delete (char *name); -char *concat (char *s1, char *s2, char *s3); -long *xmalloc (unsigned int size); +static void fatal (char *s1, char *s2, char *s3); +static void error (char *s1, char *s2, char *s3); +static void pfatal_with_name (char *name); +static void pfatal_and_delete (char *name); +static char *concat (char *s1, char *s2, char *s3); +static long *xmalloc (unsigned int size); #ifdef MAIL_USE_POP -int popmail (char *mailbox, char *outfile, int preserve, char *password, int reverse_order); -int pop_retr (popserver server, int msgno, FILE *arg); -int mbx_write (char *line, int len, FILE *mbf); -int mbx_delimit_begin (FILE *mbf); -int mbx_delimit_end (FILE *mbf); +static int popmail (char *mailbox, char *outfile, int preserve, char *password, int reverse_order); +static int pop_retr (popserver server, int msgno, FILE *arg); +static int mbx_write (char *line, int len, FILE *mbf); +static int mbx_delimit_begin (FILE *mbf); +static int mbx_delimit_end (FILE *mbf); #endif /* Nonzero means this is name of a lock file to delete on fatal error. */ @@ -589,7 +594,7 @@ /* Print error message and exit. */ -void +static void fatal (char *s1, char *s2, char *s3) { if (delete_lockname) @@ -601,7 +606,7 @@ /* Print error message. `s1' is printf control string, `s2' and `s3' are args for it or null. */ -void +static void error (char *s1, char *s2, char *s3) { fprintf (stderr, "movemail: "); @@ -614,13 +619,13 @@ fprintf (stderr, "\n"); } -void +static void pfatal_with_name (char *name) { fatal ("%s for %s", strerror (errno), name); } -void +static void pfatal_and_delete (char *name) { char *s = strerror (errno); @@ -630,7 +635,7 @@ /* Return a newly-allocated string whose contents concatenate those of s1, s2, s3. */ -char * +static char * concat (char *s1, char *s2, char *s3) { int len1 = strlen (s1), len2 = strlen (s2), len3 = strlen (s3); @@ -646,7 +651,7 @@ /* Like malloc but get fatal error if memory is exhausted. */ -long * +static long * xmalloc (unsigned int size) { long *result = (long *) malloc (size); @@ -696,7 +701,7 @@ * Return a value suitable for passing to `exit'. */ -int +static int popmail (char *mailbox, char *outfile, int preserve, char *password, int reverse_order) { int nmsgs, nbytes; @@ -822,10 +827,9 @@ return EXIT_SUCCESS; } -int +static int pop_retr (popserver server, int msgno, FILE *arg) { - extern char *strerror (int); char *line; int ret; @@ -870,7 +874,7 @@ && (a[3] == 'm') \ && (a[4] == ' ')) -int +static int mbx_write (char *line, int len, FILE *mbf) { #ifdef MOVEMAIL_QUOTE_POP_FROM_LINES @@ -894,7 +898,7 @@ return (OK); } -int +static int mbx_delimit_begin (FILE *mbf) { time_t now; @@ -911,7 +915,7 @@ return (OK); } -int +static int mbx_delimit_end (FILE *mbf) { if (putc ('\n', mbf) == EOF)