[exim-cvs] Remove attempts to quieten compiler static-checki…

Gitweb: https://git.exim.org/exim.git/commitdiff/2bde51964df7b459f1cc6853ffa7f5466d02554f

Commit:     2bde51964df7b459f1cc6853ffa7f5466d02554f
Parent:     f15b0e22422af2e14091e0ac6a58454b6849cba6
Author:     Jeremy Harris <jgh146exb@???>
AuthorDate: Sun Jun 28 14:16:20 2020 +0100
Committer:  Jeremy Harris <jgh146exb@???>
CommitDate: Sun Jun 28 14:16:20 2020 +0100

        Remove attempts to quieten compiler static-checking (more)
---
 src/src/acl.c               |  2 --
 src/src/auths/auth-spa.c    |  2 --
 src/src/dbfn.c              |  1 -
 src/src/debug.c             |  2 +-
 src/src/dns.c               |  3 +--
 src/src/exim.c              | 12 +++++-------
 src/src/exim_dbutil.c       |  1 -
 src/src/filtertest.c        |  2 +-
 src/src/ip.c                |  4 ----
 src/src/log.c               |  5 +----
 src/src/match.c             |  4 ----
 src/src/routers/ipliteral.c |  1 -
 src/src/smtp_in.c           |  4 +---
 src/src/spool_mbox.c        |  3 +--
 src/src/tls-openssl.c       |  1 -
 src/src/transport.c         |  1 -
 src/src/transports/pipe.c   |  7 -------
 src/src/transports/smtp.c   |  1 -
 18 files changed, 11 insertions(+), 45 deletions(-)

diff --git a/src/src/acl.c b/src/src/acl.c
index 1bb0cbb..8e17a0e 100644
--- a/src/src/acl.c
+++ b/src/src/acl.c
@@ -1186,8 +1186,6 @@ acl_verify_reverse(uschar **user_msgptr, uschar **log_msgptr)
{
int rc;

-user_msgptr = user_msgptr; /* stop compiler warning */
-
/* Previous success */

if (sender_host_name != NULL) return OK;
diff --git a/src/src/auths/auth-spa.c b/src/src/auths/auth-spa.c
index 44c99e9..583a2b9 100644
--- a/src/src/auths/auth-spa.c
+++ b/src/src/auths/auth-spa.c
@@ -1395,8 +1395,6 @@ int i;
int p = (int)getpid();
int random_seed = (int)time(NULL) ^ ((p << 16) | p);

-request = request; /* Added by PH to stop compilers whinging */
-
/* Ensure challenge data is cleared, in case it isn't all used. This
patch added by PH on suggestion of Russell King */

diff --git a/src/src/dbfn.c b/src/src/dbfn.c
index bbf20a1..aa08320 100644
--- a/src/src/dbfn.c
+++ b/src/src/dbfn.c
@@ -423,7 +423,6 @@ dbfn_scan(open_db *dbblock, BOOL start, EXIM_CURSOR **cursor)
 {
 EXIM_DATUM key_datum, value_datum;
 uschar *yield;
-value_datum = value_datum;    /* dummy; not all db libraries use this */

DEBUG(D_hints_lookup) debug_printf_indent("dbfn_scan\n");

diff --git a/src/src/debug.c b/src/src/debug.c
index 3a7d6a6..44e0cee 100644
--- a/src/src/debug.c
+++ b/src/src/debug.c
@@ -332,7 +332,7 @@ if (fstat(fd, &s) == 0 && (s.st_mode & S_IFMT) == S_IFSOCK)
socklen_t alen = sizeof(a);
struct sockaddr_in * sinp = (struct sockaddr_in *)&a;
struct sockaddr_in6 * sin6p = (struct sockaddr_in6 *)&a;
- struct sockaddr_un * sa_unp ; (struct sockaddr_un *)&a;
+ struct sockaddr_un * sa_unp = (struct sockaddr_un *)&a;

   if (getsockname(fd, &a, &alen) == 0)
     switch (sinp->sin_family)
diff --git a/src/src/dns.c b/src/src/dns.c
index b567c3e..98c44b9 100644
--- a/src/src/dns.c
+++ b/src/src/dns.c
@@ -334,7 +334,6 @@ char * trace = NULL;
 #ifdef rr_trace
 # define TRACE DEBUG(D_dns)
 #else
-trace = trace;
 # define TRACE if (FALSE)
 #endif

@@ -1249,7 +1248,7 @@ switch (type)

     /* Extract the numerical SRV fields (p is incremented) */
     GETSHORT(priority, p);
-    GETSHORT(weight, p);    weight = weight; /* compiler quietening */
+    GETSHORT(weight, p);
     GETSHORT(port, p);

     /* Check the CSA version number */
diff --git a/src/src/exim.c b/src/src/exim.c
index dcfc066..9d3d456 100644
--- a/src/src/exim.c
+++ b/src/src/exim.c
@@ -53,7 +53,7 @@ return store_get((int)size, TRUE);
 }

static void
-function_dummy_free(void *block) { block = block; }
+function_dummy_free(void * block) {}

static void *
function_store_malloc(size_t size)
@@ -62,7 +62,7 @@ return store_malloc((int)size);
}

static void
-function_store_free(void *block)
+function_store_free(void * block)
{
store_free(block);
}
@@ -4101,10 +4101,8 @@ privilege by now. Before the chdir, we try to ensure that the directory exists.

 if (Uchdir(spool_directory) != 0)
   {
-  int dummy;
-  (void)directory_make(spool_directory, US"", SPOOL_DIRECTORY_MODE, FALSE);
-  dummy = /* quieten compiler */ Uchdir(spool_directory);
-  dummy = dummy;    /* yet more compiler quietening, sigh */
+  (void) directory_make(spool_directory, US"", SPOOL_DIRECTORY_MODE, FALSE);
+  (void) Uchdir(spool_directory);
   }

/* Handle calls with the -bi option. This is a sendmail option to rebuild *the*
@@ -4115,7 +4113,7 @@ script. */

 if (bi_option)
   {
-  (void)fclose(config_file);
+  (void) fclose(config_file);
   if (bi_command && *bi_command)
     {
     int i = 0;
diff --git a/src/src/exim_dbutil.c b/src/src/exim_dbutil.c
index 8513bc0..7429527 100644
--- a/src/src/exim_dbutil.c
+++ b/src/src/exim_dbutil.c
@@ -515,7 +515,6 @@ dbfn_scan(open_db *dbblock, BOOL start, EXIM_CURSOR **cursor)
 {
 EXIM_DATUM key_datum, value_datum;
 uschar *yield;
-value_datum = value_datum;    /* dummy; not all db libraries use this */

/* Some dbm require an initialization */

diff --git a/src/src/filtertest.c b/src/src/filtertest.c
index f54cbef..a2a60a8 100644
--- a/src/src/filtertest.c
+++ b/src/src/filtertest.c
@@ -95,7 +95,7 @@ if (!dot_ended && !feof(stdin))
       if (s > message_body_end + message_body_visible) s = message_body_end;
       message_size++;
       }
-    READ_END: ch = ch;  /* Some compilers don't like null statements */
+    READ_END: ;
     }
   if (s == message_body_end || s[-1] != '\n') body_linecount++;
   }
diff --git a/src/src/ip.c b/src/src/ip.c
index a5958e9..c09962a 100644
--- a/src/src/ip.c
+++ b/src/src/ip.c
@@ -127,8 +127,6 @@ if (af == AF_INET6)
   return sizeof(sin->v6);
   }
 else
-#else     /* HAVE_IPv6 */
-af = af;  /* Avoid compiler warning */
 #endif    /* HAVE_IPV6 */

 /* Setup code when using IPv4 socket. The wildcard address is "". */
@@ -209,8 +207,6 @@ if (af == AF_INET6)
   s_len = sizeof(s_in6);
   }
 else
-#else     /* HAVE_IPV6 */
-af = af;  /* Avoid compiler warning */
 #endif    /* HAVE_IPV6 */

 /* For an IPv4 address, use an IPv4 sockaddr structure, even on a system with
diff --git a/src/src/log.c b/src/src/log.c
index 99eba5f..9f3f45e 100644
--- a/src/src/log.c
+++ b/src/src/log.c
@@ -1188,10 +1188,7 @@ if (flags & LOG_PANIC)
     panic_recurseflag = FALSE;

     if (panic_save_buffer)
-      {
-      int i = write(paniclogfd, panic_save_buffer, Ustrlen(panic_save_buffer));
-      i = i;    /* compiler quietening */
-      }
+      (void) write(paniclogfd, panic_save_buffer, Ustrlen(panic_save_buffer));

     written_len = write_to_fd_buf(paniclogfd, g->s, g->ptr);
     if (written_len != g->ptr)
diff --git a/src/src/match.c b/src/src/match.c
index 2f6c922..18de977 100644
--- a/src/src/match.c
+++ b/src/src/match.c
@@ -104,8 +104,6 @@ uschar *filename = NULL;
 uschar *keyquery, *result, *semicolon;
 void *handle;

-error = error; /* Keep clever compilers from complaining */
-
if (valueptr) *valueptr = NULL;

/* For regular expressions, use cb->origsubject rather than cb->subject so that
@@ -1004,8 +1002,6 @@ uschar *subject = cb->address;
const uschar *s;
uschar *pdomain, *sdomain;

-error = error; /* Keep clever compilers from complaining */
-
DEBUG(D_lists) debug_printf("address match test: subject=%s pattern=%s\n",
subject, pattern);

diff --git a/src/src/routers/ipliteral.c b/src/src/routers/ipliteral.c
index f0b3b19..09775ed 100644
--- a/src/src/routers/ipliteral.c
+++ b/src/src/routers/ipliteral.c
@@ -57,7 +57,6 @@ ipliteral_router_init(router_instance *rblock)
ipliteral_router_options_block *ob =
(ipliteral_router_options_block *)(rblock->options_block);
*/
-rblock = rblock;
}

diff --git a/src/src/smtp_in.c b/src/src/smtp_in.c
index 39f5199..ef3964f 100644
--- a/src/src/smtp_in.c
+++ b/src/src/smtp_in.c
@@ -4497,10 +4497,8 @@ while (done <= 0)
 # endif
       else
 #endif
+    (void) fwrite(g->s, 1, g->ptr, smtp_out);

-    {
-    int i = fwrite(g->s, 1, g->ptr, smtp_out); i = i; /* compiler quietening */
-    }
       DEBUG(D_receive)
     {
     uschar *cr;
diff --git a/src/src/spool_mbox.c b/src/src/spool_mbox.c
index 4f3c4d2..9f1efbf 100644
--- a/src/src/spool_mbox.c
+++ b/src/src/spool_mbox.c
@@ -226,12 +226,11 @@ if (spool_mbox_ok && !f.no_mbox_unspool)
   for (struct dirent *entry; entry = readdir(tempdir); )
     {
     uschar *name = US entry->d_name;
-    int dummy;
     if (Ustrcmp(name, US".") == 0 || Ustrcmp(name, US"..") == 0) continue;

     file_path = string_sprintf("%s/%s", mbox_path, name);
     debug_printf("unspool_mbox(): unlinking '%s'\n", file_path);
-    dummy = unlink(CS file_path); dummy = dummy;    /* compiler quietening */
+    (void) unlink(CS file_path);
     }

closedir(tempdir);
diff --git a/src/src/tls-openssl.c b/src/src/tls-openssl.c
index 525afd6..50384cb 100644
--- a/src/src/tls-openssl.c
+++ b/src/src/tls-openssl.c
@@ -496,7 +496,6 @@ RSA *rsa_key;
BIGNUM *bn = BN_new();
#endif

-export = export;     /* Shut picky compilers up */
 DEBUG(D_tls) debug_printf("Generating %d bit RSA key...\n", keylength);

 #ifdef EXIM_HAVE_RSA_GENKEY_EX
diff --git a/src/src/transport.c b/src/src/transport.c
index f9ff521..aed743d 100644
--- a/src/src/transport.c
+++ b/src/src/transport.c
@@ -1400,7 +1400,6 @@ if (write_pid > 0)
     int dummy = read(pfd[pipe_read], (void *)&save_errno, sizeof(int));
         dummy = read(pfd[pipe_read], (void *)&tctx->addr->more_errno, sizeof(int));
         dummy = read(pfd[pipe_read], (void *)&tctx->addr->delivery_time, sizeof(struct timeval));
-    dummy = dummy;        /* compiler quietening */
         yield = FALSE;
         }
       }
diff --git a/src/src/transports/pipe.c b/src/src/transports/pipe.c
index 27422bd..1cb574e 100644
--- a/src/src/transports/pipe.c
+++ b/src/src/transports/pipe.c
@@ -144,13 +144,6 @@ pipe_transport_setup(transport_instance *tblock, address_item *addrlist,
 pipe_transport_options_block *ob =
   (pipe_transport_options_block *)(tblock->options_block);

-addrlist = addrlist;  /* Keep compiler happy */
-dummy = dummy;
-uid = uid;
-gid = gid;
-errmsg = errmsg;
-ob = ob;
-
 #ifdef HAVE_SETCLASSRESOURCES
 if (ob->use_classresources)
   {
diff --git a/src/src/transports/smtp.c b/src/src/transports/smtp.c
index 0f3ade9..46663b0 100644
--- a/src/src/transports/smtp.c
+++ b/src/src/transports/smtp.c
@@ -3428,7 +3428,6 @@ uschar *message = NULL;
 uschar new_message_id[MESSAGE_ID_LENGTH + 1];
 smtp_context * sx = store_get(sizeof(*sx), TRUE);    /* tainted, for the data buffers */

-suppress_tls = suppress_tls; /* stop compiler warning when no TLS support */
*message_defer = FALSE;

memset(sx, 0, sizeof(*sx));