Fixed some more problems caught by gcc -ansi -pedantic.

diff --git a/mts/smtp/smtp.c b/mts/smtp/smtp.c
index ba9938a47edf4f2083679b14b4ef03fe2473c956..c18b89b1671b4d3454be72730527664e61dcaf97 100644 (file)
--- a/mts/smtp/smtp.c
+++ b/mts/smtp/smtp.c
@@ -1115,12 +1115,12 @@ static int
 sm_get_pass(sasl_conn_t *conn, void *context, int id,
            sasl_secret_t **psecret)
 {
 sm_get_pass(sasl_conn_t *conn, void *context, int id,
            sasl_secret_t **psecret)
 {

-    NMH_UNUSED (conn);
-

     char **pw_context = (char **) context;
     char *pass = NULL;
     int len;
 
     char **pw_context = (char **) context;
     char *pass = NULL;
     int len;
 

+    NMH_UNUSED (conn);
+

     if (! psecret || id != SASL_CB_PASS)
        return SASL_BADPARAM;
 
     if (! psecret || id != SASL_CB_PASS)
        return SASL_BADPARAM;
 

diff --git a/sbr/getansreadline.c b/sbr/getansreadline.c
index 26a0574110da7a5d2950fc3e7249f5e91ffa9eff..2fd06186d29a0ac890a3459e3b999df9616f82d0 100644 (file)
--- a/sbr/getansreadline.c
+++ b/sbr/getansreadline.c
@@ -175,10 +175,10 @@ initialize_readline(void)
 static char **
 nmh_completion(const char *text, int start, int end)
 {
 static char **
 nmh_completion(const char *text, int start, int end)
 {

-    NMH_UNUSED (end);
-

     char **matches;
 
     char **matches;
 

+    NMH_UNUSED (end);
+

     matches = (char **) NULL;
 
     if (start == 0)
     matches = (char **) NULL;
 
     if (start == 0)

diff --git a/sbr/m_mktemp.c b/sbr/m_mktemp.c
index 9f99119e1bb04fe88a5b8ad8894f862d596c346f..c08f614e1fc5c25c4eb685df5b356b9c1a851e3c 100644 (file)
--- a/sbr/m_mktemp.c
+++ b/sbr/m_mktemp.c
@@ -130,7 +130,7 @@ m_mktemp2 (
 static char *
 get_temp_dir()
 {
 static char *
 get_temp_dir()
 {

-    // Ignore envvars if we are setuid
+    /* Ignore envvars if we are setuid */

     if ((getuid()==geteuid()) && (getgid()==getegid())) {
         char *tmpdir = NULL;
         tmpdir = getenv("MHTMPDIR");
     if ((getuid()==geteuid()) && (getgid()==getegid())) {
         char *tmpdir = NULL;
         tmpdir = getenv("MHTMPDIR");

diff --git a/uip/mhparse.c b/uip/mhparse.c
index 803511365c789ac99d443484f0e0973193686712..4c3a19460b0cdfe27214b23568e651457bf28e3e 100644 (file)
--- a/uip/mhparse.c
+++ b/uip/mhparse.c
@@ -1776,8 +1776,8 @@ openBase64 (CT ct, char **file)
     }
     if (cp != NULL && *cp != '\0') {
         if (ce->ce_unlink) {
     }
     if (cp != NULL && *cp != '\0') {
         if (ce->ce_unlink) {

-            // Temporary file already exists, so we rename to
-            // version with extension.
+            /* Temporary file already exists, so we rename to
+               version with extension. */

             char *file_org = strdup(ce->ce_file);
             ce->ce_file = add (cp, ce->ce_file);
             if (rename(file_org, ce->ce_file)) {
             char *file_org = strdup(ce->ce_file);
             ce->ce_file = add (cp, ce->ce_file);
             if (rename(file_org, ce->ce_file)) {


@@ -2008,8 +2008,8 @@ openQuoted (CT ct, char **file)
     }
     if (cp != NULL && *cp != '\0') {
         if (ce->ce_unlink) {
     }
     if (cp != NULL && *cp != '\0') {
         if (ce->ce_unlink) {

-            // Temporary file already exists, so we rename to
-            // version with extension.
+            /* Temporary file already exists, so we rename to
+               version with extension. */

             char *file_org = strdup(ce->ce_file);
             ce->ce_file = add (cp, ce->ce_file);
             if (rename(file_org, ce->ce_file)) {
             char *file_org = strdup(ce->ce_file);
             ce->ce_file = add (cp, ce->ce_file);
             if (rename(file_org, ce->ce_file)) {


@@ -2232,8 +2232,8 @@ open7Bit (CT ct, char **file)
     }
     if (cp != NULL && *cp != '\0') {
         if (ce->ce_unlink) {
     }
     if (cp != NULL && *cp != '\0') {
         if (ce->ce_unlink) {

-            // Temporary file already exists, so we rename to
-            // version with extension.
+            /* Temporary file already exists, so we rename to
+               version with extension. */

             char *file_org = strdup(ce->ce_file);
             ce->ce_file = add (cp, ce->ce_file);
             if (rename(file_org, ce->ce_file)) {
             char *file_org = strdup(ce->ce_file);
             ce->ce_file = add (cp, ce->ce_file);
             if (rename(file_org, ce->ce_file)) {

diff --git a/uip/picksbr.c b/uip/picksbr.c
index b92c053a8bf07391cf24dd28f120bf89a5622d29..4073919d61835e16f6b5313a15ca8073757f0759 100644 (file)
--- a/uip/picksbr.c
+++ b/uip/picksbr.c
@@ -80,7 +80,7 @@ static struct swit parswit[] = {
 static char linebuf[LBSIZE + 1];
 
 /* the magic array for case-independence */
 static char linebuf[LBSIZE + 1];
 
 /* the magic array for case-independence */

-static char cc[] = {
+static unsigned char cc[] = {

        0000,0001,0002,0003,0004,0005,0006,0007,
        0010,0011,0012,0013,0014,0015,0016,0017,
        0020,0021,0022,0023,0024,0025,0026,0027,
        0000,0001,0002,0003,0004,0005,0006,0007,
        0010,0011,0012,0013,0014,0015,0016,0017,
        0020,0021,0022,0023,0024,0025,0026,0027,

diff --git a/uip/popsbr.c b/uip/popsbr.c
index ff28705f07af58d4b3d931042279aaaca0f7f886..c566264d3f0f7ea40ee0dbe7bd53583d5d2d5a6f 100644 (file)
--- a/uip/popsbr.c
+++ b/uip/popsbr.c
@@ -343,12 +343,12 @@ sasl_get_user(void *context, int id, const char **result, unsigned *len)
 static int
 sasl_get_pass(sasl_conn_t *conn, void *context, int id, sasl_secret_t **psecret)
 {
 static int
 sasl_get_pass(sasl_conn_t *conn, void *context, int id, sasl_secret_t **psecret)
 {

-    NMH_UNUSED (conn);
-

     struct pass_context *p_context = (struct pass_context *) context;
     char *pass = NULL;
     int len;
 
     struct pass_context *p_context = (struct pass_context *) context;
     char *pass = NULL;
     int len;
 

+    NMH_UNUSED (conn);
+

     if (! psecret || id != SASL_CB_PASS)
        return SASL_BADPARAM;
 
     if (! psecret || id != SASL_CB_PASS)
        return SASL_BADPARAM;
 

diff --git a/uip/sendsbr.c b/uip/sendsbr.c
index d866ebc925b13a75dfdd513894b5d1f7b52cf426..3eff75d97f803cba60fea318a43a26ac2649d390 100644 (file)
--- a/uip/sendsbr.c
+++ b/uip/sendsbr.c
@@ -502,10 +502,12 @@ make_mime_composition_file_entry(char *file_name, int attachformat)
         } else {
             /* Suppress Content-Id, insert simple Content-Disposition
                and Content-Description with filename. */
         } else {
             /* Suppress Content-Id, insert simple Content-Disposition
                and Content-Description with filename. */

+            p = strrchr(file_name, '/');

             (void) fprintf (composition_file,
             (void) fprintf (composition_file,

-                            "#%1$s; name=\"%2$s\" <> [%2$s]{attachment}",
+                            "#%s; name=\"%s\" <> [%s]{attachment}",

                             content_type,
                             content_type,

-                            ((p = strrchr(file_name, '/')) == (char *)0) ? file_name : p + 1);
+                            (p == (char *)0) ? file_name : p + 1,
+                            (p == (char *)0) ? file_name : p + 1);

         }
 
         break;
         }
 
         break;


@@ -519,10 +521,12 @@ make_mime_composition_file_entry(char *file_name, int attachformat)
         } else {
             /* Suppress Content-Id, insert Content-Disposition with
                modification date and Content-Description wtih filename. */
         } else {
             /* Suppress Content-Id, insert Content-Disposition with
                modification date and Content-Description wtih filename. */

+            p = strrchr(file_name, '/');

             (void) fprintf (composition_file,
             (void) fprintf (composition_file,

-                            "#%1$s; name=\"%2$s\" <>[%2$s]{attachment; modification-date=\"%3$s\"}",
+                            "#%s; name=\"%s\" <>[%s]{attachment; modification-date=\"%s\"}",

                             content_type,
                             content_type,

-                            ((p = strrchr(file_name, '/')) == (char *)0) ? file_name : p + 1,
+                            (p == (char *)0) ? file_name : p + 1,
+                            (p == (char *)0) ? file_name : p + 1,

                             dtime (&st.st_mtime, 0));
         }
 
                             dtime (&st.st_mtime, 0));
         }