[exim-cvs] tidying

Gitweb: https://git.exim.org/exim.git/commitdiff/ccdaab638bb794bbf9f7e7c59f08892ef585f8c6

Commit:     ccdaab638bb794bbf9f7e7c59f08892ef585f8c6
Parent:     1e1e2dec3570628a7d01fc0d3b0ef1bba7ac04fd
Author:     Jeremy Harris <jgh146exb@???>
AuthorDate: Tue Nov 26 19:07:41 2024 +0000
Committer:  Jeremy Harris <jgh146exb@???>
CommitDate: Tue Nov 26 21:42:47 2024 +0000

    tidying
---
 src/exim_monitor/em_version.c |  2 ++
 src/src/buildconfig.c         | 40 +++++++++++-----------------------------
 src/src/exim.c                |  2 +-
 src/src/spool_in.c            |  2 ++
 4 files changed, 16 insertions(+), 30 deletions(-)

diff --git a/src/exim_monitor/em_version.c b/src/exim_monitor/em_version.c
index 066ccab56..8f047ce5a 100644
--- a/src/exim_monitor/em_version.c
+++ b/src/exim_monitor/em_version.c
@@ -24,8 +24,10 @@ extern uschar *version_date;
void
version_init(void)
{
+#ifndef EXIM_BUILD_DATE_OVERRIDE
int i = 0;
uschar today[20];
+#endif

version_string = US"2.06";

diff --git a/src/src/buildconfig.c b/src/src/buildconfig.c
index e594bf6f2..367420095 100644
--- a/src/src/buildconfig.c
+++ b/src/src/buildconfig.c
@@ -103,27 +103,9 @@ if (!OK)
int
main(int argc, char **argv)
{
-off_t test_off_t = 0;
-time_t test_time_t = 0;
-ino_t test_ino_t;
-#if ! (__STDC_VERSION__ >= 199901L)
-size_t test_size_t = 0;
-ssize_t test_ssize_t = 0;
-unsigned long test_ulong_t = 0L;
-unsigned int test_uint_t = 0;
-#endif
-long test_long_t = 0;
-long long test_longlong_t = 0;
-int test_int_t = 0;
-FILE *base;
-FILE *new;
-int last_initial = 'A';
-int linecount = 0;
-int have_auth = 0;
-int in_local_makefile = 0;
-int use_which_db = 0;
-int use_which_db_in_local_makefile = 0;
-int support_crypteq = 0;
+FILE * base, * new;
+int last_initial = 'A', linecount = 0, have_auth = 0, in_local_makefile = 0;
+int use_which_db = 0, use_which_db_in_local_makefile = 0, support_crypteq = 0;
char buffer[1024];

if (argc != 1)
@@ -157,14 +139,14 @@ printing long long variables, and there will be support for the long long type.
This assumption is known to be OK for the common operating systems. */

fprintf(new, "#ifndef OFF_T_FMT\n");
-if (sizeof(test_off_t) > sizeof(test_long_t))
+if (sizeof(off_t) > sizeof(long))
fprintf(new, "# define OFF_T_FMT \"%%lld\"\n");
else
fprintf(new, "# define OFF_T_FMT \"%%ld\"\n");
fprintf(new, "#endif\n\n");

fprintf(new, "#ifndef LONGLONG_T\n");
-if (sizeof(test_longlong_t) > sizeof(test_long_t))
+if (sizeof(long long) > sizeof(long))
fprintf(new, "# define LONGLONG_T long long int\n");
else
fprintf(new, "# define LONGLONG_T long int\n");
@@ -176,7 +158,7 @@ length is 4 or less, we can leave LONGLONG_T to whatever was defined above for
off_t. */

fprintf(new, "#ifndef TIME_T_FMT\n");
-if (sizeof(test_time_t) > sizeof(test_long_t))
+if (sizeof(time_t) > sizeof(long))
{
fprintf(new, "# define TIME_T_FMT \"%%lld\"\n");
fprintf(new, "# undef LONGLONG_T\n");
@@ -187,7 +169,7 @@ else
fprintf(new, "#endif\n\n");

fprintf(new, "#ifndef INO_T_FMT\n");
-if (sizeof(test_ino_t) > sizeof(test_long_t))
+if (sizeof(ino_t) > sizeof(long))
fprintf(new, "# define INO_T_FMT \"%%llu\"\n");
else
fprintf(new, "# define INO_T_FMT \"%%lu\"\n");
@@ -206,16 +188,16 @@ it then. */
fprintf(new, "#define SIZE_T_FMT \"%%zu\"\n");
fprintf(new, "#define SSIZE_T_FMT \"%%zd\"\n");
#else
-if (sizeof(test_size_t) > sizeof (test_ulong_t))
+if (sizeof(size_t) > sizeof (unsigned long))
fprintf(new, "#define SIZE_T_FMT \"%%llu\"\n");
-else if (sizeof(test_size_t) > sizeof (test_uint_t))
+else if (sizeof(size_t) > sizeof (unsigned int))
fprintf(new, "#define SIZE_T_FMT \"%%lu\"\n");
else
fprintf(new, "#define SIZE_T_FMT \"%%u\"\n");

-if (sizeof(test_ssize_t) > sizeof(test_long_t))
+if (sizeof(ssize_t) > sizeof(long))
   fprintf(new, "#define SSIZE_T_FMT  \"%%lld\"\n");
-else if (sizeof(test_ssize_t) > sizeof(test_int_t))
+else if (sizeof(ssize_t) > sizeof(int))
   fprintf(new, "#define SSIZE_T_FMT  \"%%ld\"\n");
 else
   fprintf(new, "#define SSIZE_T_FMT  \"%%d\"\n");
diff --git a/src/src/exim.c b/src/src/exim.c
index 49bbf7682..2297a5edc 100644
--- a/src/src/exim.c
+++ b/src/src/exim.c
@@ -4625,7 +4625,7 @@ privilege by now. Before the chdir, we try to ensure that the directory exists.
 if (Uchdir(spool_directory) != 0)
   {
   (void) directory_make(spool_directory, US"", SPOOL_DIRECTORY_MODE, FALSE);
-  (void) Uchdir(spool_directory);
+  (void) Uchdir(spool_directory);    /*XXX maybe panic on fail? */
   }

 /* Handle calls with the -bi option. This is a sendmail option to rebuild *the*
diff --git a/src/src/spool_in.c b/src/src/spool_in.c
index 43b30986d..ab24c20db 100644
--- a/src/src/spool_in.c
+++ b/src/src/spool_in.c
@@ -1075,6 +1075,8 @@ SPOOL_FORMAT_ERROR:
 #ifndef COMPILE_UTILITY
 DEBUG(D_any) debug_printf("Format error in spool file %s%s%s\n", name,
   where ? ": " : "", where ? where : US"");
+#else
+where = where;    /* compiler quietening */
 #endif  /* COMPILE_UTILITY */

fclose(fp);

--
## subscription configuration (requires account):
## https://lists.exim.org/mailman3/postorius/lists/exim-cvs.lists.exim.org/
## unsubscribe (doesn't require an account):
## exim-cvs-unsubscribe@???
## Exim details at http://www.exim.org/
## Please use the Wiki with this list - http://wiki.exim.org/