Fix gcc warnings when building for Windows.
authorWerner Koch <wk@gnupg.org>
Wed, 20 Feb 2019 12:14:46 +0000 (13:14 +0100)
committerWerner Koch <wk@gnupg.org>
Wed, 20 Feb 2019 12:17:03 +0000 (13:17 +0100)
* m4/estream.m4: Remove file.
* configure.ac (estream_PRINTF_INIT): Remove.  Estream is part of
libgpg-error.
(AC_CHECK_FUNCS): Move to a later place.
* src/agent.c [W32]: Explictly include winsock2.h to avoid a warning.
(gnupg_allow_set_foregound_window): Fix printf warning.
* src/support.h (ttyname): Mark unsused arg.
(stpcpy): Move to ...
* src/get-path.c (my_stpcpy): here.  Use macros to get around warning
pertaining to the gcc built-in.
(dlopen): Mark unused arg.

Signed-off-by: Werner Koch <wk@gnupg.org>
configure.ac
m4/Makefile.am
m4/estream.m4 [deleted file]
src/agent.c
src/get-path.c
src/p11-finalize.c
src/p11-initialize.c
src/support.h
tests/t-getslotinfo.c

index 9b0f4fd..e36f56d 100644 (file)
@@ -262,13 +262,6 @@ AC_SUBST(BUILD_REVISION_DEC)
 AC_SUBST(BUILD_TIMESTAMP)
 AC_SUBST(BUILD_FILEVERSION)
 
-# Checks for libraries.
-
-AC_CHECK_FUNCS([ttyname localtime_r timegm stpcpy])
-
-# Run the checks needed for estream-printf.c
-estream_PRINTF_INIT
-
 # The error code library.  Error codes are sent over the IPC layer and
 # have to be interpreted.
 AM_PATH_GPG_ERROR("$NEED_GPG_ERROR_VERSION",
@@ -416,6 +409,7 @@ AC_HEADER_STDBOOL
 AC_C_INLINE
 
 # Checks for library functions.
+AC_CHECK_FUNCS([ttyname localtime_r timegm stpcpy])
 
 # Check for programs needed for the manual.
 AC_CHECK_PROG(CONVERT, convert, convert)
index bff7b15..cbcd9a8 100644 (file)
@@ -1,2 +1,2 @@
-EXTRA_DIST = estream.m4 autobuild.m4 gpg-error.m4 libassuan.m4 libtool.m4 \
+EXTRA_DIST = autobuild.m4 gpg-error.m4 libassuan.m4 libtool.m4 \
              ltoptions.m4 ltsugar.m4 ltversion.m4 lt~obsolete.m4
diff --git a/m4/estream.m4 b/m4/estream.m4
deleted file mode 100644 (file)
index 4b6c745..0000000
+++ /dev/null
@@ -1,48 +0,0 @@
-dnl Autoconf macros for libestream
-dnl       Copyright (C) 2007 g10 Code GmbH
-dnl
-dnl This file is free software; as a special exception the author gives
-dnl unlimited permission to copy and/or distribute it, with or without
-dnl modifications, as long as this notice is preserved.
-dnl
-dnl This file is distributed in the hope that it will be useful, but
-dnl WITHOUT ANY WARRANTY, to the extent permitted by law; without even the
-dnl implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
-
-
-dnl estream_PRINTF_INIT
-dnl Prepare build of source included estream-printf.c
-dnl
-AC_DEFUN([estream_PRINTF_INIT],
-[ 
-  AC_MSG_NOTICE([checking system features for estream-printf])
-  AC_TYPE_LONG_LONG_INT  
-  AC_TYPE_LONG_DOUBLE  
-  AC_TYPE_INTMAX_T
-  AC_TYPE_UINTMAX_T
-  AC_CHECK_TYPES([ptrdiff_t])
-  AC_CHECK_SIZEOF([unsigned long])
-  AC_CHECK_SIZEOF([void *])
-  AC_CACHE_CHECK([for nl_langinfo and THOUSANDS_SEP],
-                  estream_cv_langinfo_thousands_sep,
-      [AC_TRY_LINK([#include <langinfo.h>],
-        [char* cs = nl_langinfo(THOUSANDS_SEP); return !cs;],
-        estream_cv_langinfo_thousands_sep=yes,
-        estream_cv_langinfo_thousands_sep=no)
-      ])
-  if test $estream_cv_langinfo_thousands_sep = yes; then
-    AC_DEFINE(HAVE_LANGINFO_THOUSANDS_SEP, 1,
-      [Define if you have <langinfo.h> and nl_langinfo(THOUSANDS_SEP).])
-  fi
-])
-
-
-dnl estream_INIT
-dnl Prepare build of source included estream.c
-dnl
-AC_DEFUN([estream_INIT],
-[ 
-  AC_REQUIRE([estream_PRINTF_INIT])
-  AC_MSG_NOTICE([checking system features for estream])
-
-])
index df9cdc1..6228af9 100644 (file)
 #include <stdarg.h>
 
 #ifdef HAVE_W32_SYSTEM
-#define PATHSEP_C ';'
-#define WINVER 0x0500  /* Required for AllowSetForegroundWindow.  */
-#include <windows.h>
+# define PATHSEP_C ';'
+# define WINVER 0x0500  /* Required for AllowSetForegroundWindow.  */
+# include <winsock2.h>
+# include <windows.h>
 #else
-#define PATHSEP_C ':'
+# define PATHSEP_C ':'
 #endif
 
 #include <assuan.h>
@@ -64,8 +65,8 @@ gnupg_allow_set_foregound_window (pid_t pid)
     return;
 #ifdef HAVE_W32_SYSTEM
   else if (!AllowSetForegroundWindow (pid))
-    DEBUG (DBG_CRIT, "AllowSetForegroundWindow(%lu) failed: %i\n",
-          (unsigned long)pid, GetLastError ());
+    DEBUG (DBG_CRIT, "AllowSetForegroundWindow(%lu) failed: %u\n",
+          (unsigned long)pid, (unsigned int)GetLastError ());
 #endif
 }
 
index 521c727..58491fe 100644 (file)
 #include <stdlib.h>
 #include <string.h>
 #ifdef HAVE_W32_SYSTEM
-#include <windows.h>
-#include <shlobj.h>
-#include <io.h>
+# include <windows.h>
+# include <shlobj.h>
+# include <io.h>
 #endif
 
 #include "support.h"
 
+#ifndef HAVE_STPCPY
+static char *
+my_stpcpy (char *a, const char *b)
+{
+    while( *b )
+       *a++ = *b++;
+    *a = 0;
+
+    return (char*)a;
+}
+# undef stpcpy
+# define stpcpy(a,b) my_stpcpy ((a), (b))
+#endif /* !HAVE_STPCPY */
+
+
+
+
 #ifdef HAVE_W32_SYSTEM
 #define RTLD_LAZY 0
 
@@ -39,6 +56,7 @@ static __inline__ void *
 dlopen (const char * name, int flag)
 {
   void * hd = LoadLibrary (name);
+  (void)flag;
   return hd;
 }
 
index 7dbd00b..834046a 100644 (file)
@@ -22,8 +22,8 @@
 #include <config.h>
 #endif
 #ifdef HAVE_W32_SYSTEM
-#define __USE_W32_SOCKETS 1
-#include <windows.h>
+# include <winsock2.h>
+# include <windows.h>
 #endif
 
 #include "cryptoki.h"
index 386df0e..6202cb5 100644 (file)
@@ -26,7 +26,8 @@
 
 #ifdef HAVE_W32_SYSTEM
 #define __USE_W32_SOCKETS 1
-#include <windows.h>
+# include <winsock2.h>
+# include <windows.h>
 #endif
 
 #include <assuan.h>
index 34f67d2..b51f3ac 100644 (file)
@@ -65,23 +65,11 @@ scute_copy_string (char *dest, const char *src, int max_len)
 static inline char *
 ttyname (int fd)
 {
+  (void)fd;
   return NULL;
 }
 #endif /* !HAVE_TTYNAME */
 
-#ifndef HAVE_STPCPY
-static inline char *
-stpcpy (char *a, const char *b)
-{
-    while( *b )
-       *a++ = *b++;
-    *a = 0;
-
-    return (char*)a;
-}
-#endif /* !HAVE_STPCPY */
-
-
 
 const char *get_gpgsm_path (void);
 const char *get_gpg_connect_agent_path (void);
index 7633e82..caf02c7 100644 (file)
@@ -146,7 +146,7 @@ main (int argc, char *argv[])
   if (loop)
     {
 #ifdef WIN32
-      _sleep (2);
+      Sleep (2*1000);
 #else
       sleep (2);  /* Why? */
 #endif