Various cleanups
authorWerner Koch <wk@gnupg.org>
Fri, 31 Oct 2003 03:46:08 +0000 (03:46 +0000)
committerWerner Koch <wk@gnupg.org>
Fri, 31 Oct 2003 03:46:08 +0000 (03:46 +0000)
ChangeLog
NEWS
cipher/ChangeLog
configure.ac
src/ChangeLog
src/ath.h
src/gcrypt.h

index 073aab4..382cc6c 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -2,7 +2,7 @@
 
        * configure.ac: Give a hint on where libgpg-error is available.
        Reformatted long lines.  Don't include gcrypt-defs.h.
-       
+       (--enable-gcc-warnings): New option.
 
 2003-10-24  Moritz Schulte  <mo@g10code.com>
 
diff --git a/NEWS b/NEWS
index 1f2934a..ad23be3 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -1,6 +1,9 @@
 Noteworthy changes in version 1.1.44 (unreleased)
 -------------------------------------------------
 
+ * Bug fixes
+
+ * Enhanced the prime API
 
  * Interface changes relative to the 1.1.43 release:
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
index aef7c57..4b0d28a 100644 (file)
@@ -6,7 +6,7 @@
 
        * random.c (gcry_random_add_bytes): Return if buflen is zero to
        avoid gcc warning about unsed parameter.
-       (MASK_LEVEL): Simplified; does not work for signed and unsigned
+       (MASK_LEVEL): Simplified; does now work for signed and unsigned
        w/o warnings.
 
        * md.c (md_start_debug): Removed the const from SUFFIX, because
index e3b9355..3b5771b 100644 (file)
@@ -624,9 +624,21 @@ AM_CONDITIONAL(CROSS_COMPILING, test x$cross_compiling = xyes)
 
 GNUPG_CHECK_GNUMAKE
 
+AC_ARG_ENABLE(gcc-warnings,
+             AC_HELP_STRING([--enable-gcc-warnings],
+                            [enable more verbose gcc warnings]),
+             [more_gcc_warnings="$enableval"],
+             [more_gcc_warnings="no"])
+
 if test "$GCC" = yes; then
-    if test "$MAINTAINER_MODE" = "yes"; then
+    if test "$USE_MAINTAINER_MODE" = "yes" ||
+       test "$more_gcc_warnings" = "yes"; then
         CFLAGS="$CFLAGS -Wall -Wcast-align -Wshadow -Wstrict-prototypes"
+        if test "$more_gcc_warnings" = "yes"; then
+            CFLAGS="$CFLAGS -W -Wpointer-arith -Wbad-function-cast"
+            CFLAGS="$CFLAGS -Wwrite-strings -Wunreachable-code"
+            CFLAGS="$CFLAGS -Wno-sign-compare"
+        fi
     else
         CFLAGS="$CFLAGS -Wall"
     fi
index 4f7ff07..541e462 100644 (file)
@@ -1,3 +1,7 @@
+2003-10-27  Werner Koch  <wk@gnupg.org>
+
+       * ath.h (ATH_MUTEX_INITIALIZER): Removed spurious semicolon.
+
 2003-10-27  Moritz Schulte  <mo@g10code.com>
 
        * libgcrypt-config.in: Include libs/cflags of libgpg-error.
index 7b575f1..b01e07a 100644 (file)
--- a/src/ath.h
+++ b/src/ath.h
 #ifndef ATH_H
 #define ATH_H
 
+#ifdef _WIN32
+#warning We need to replace these hacks by cleaner code.
+typedef int ssize_t;
+typedef int pid_t;
+#include <windows.h>
+#else
 #include <sys/types.h>
 #include <sys/socket.h>
+#endif
 
 \f
 /* Define _ATH_EXT_SYM_PREFIX if you want to give all external symbols
@@ -49,7 +56,7 @@
 
 \f
 typedef void *ath_mutex_t;
-#define ATH_MUTEX_INITIALIZER 0;
+#define ATH_MUTEX_INITIALIZER 0
 
 /* Functions for mutual exclusion.  */
 int ath_mutex_init (ath_mutex_t *mutex);
@@ -66,8 +73,10 @@ ssize_t ath_select (int nfd, fd_set *rset, fd_set *wset, fd_set *eset,
 ssize_t ath_waitpid (pid_t pid, int *status, int options);
 int ath_accept (int s, struct sockaddr *addr, socklen_t *length_ptr);
 int ath_connect (int s, struct sockaddr *addr, socklen_t length);
+#ifndef _WIN32
 int ath_sendmsg (int s, const struct msghdr *msg, int flags);
 int ath_recvmsg (int s, struct msghdr *msg, int flags);
+#endif
 
 #define _ATH_COMPAT
 #ifdef _ATH_COMPAT
@@ -84,8 +93,10 @@ struct ath_ops
   ssize_t (*waitpid) (pid_t pid, int *status, int options);
   int (*accept) (int s, struct sockaddr *addr, socklen_t *length_ptr);
   int (*connect) (int s, struct sockaddr *addr, socklen_t length);
+#ifndef WIN32
   int (*sendmsg) (int s, const struct msghdr *msg, int flags);
   int (*recvmsg) (int s, struct msghdr *msg, int flags);
+#endif
 };
 
 /* Initialize the any-thread package.  */
index 51a53e3..539e121 100644 (file)
@@ -53,7 +53,7 @@ extern "C" {
 #endif
 
 /* We want to use gcc attributes when possible.  Warning: Don't use
-   these macros in your progranms: As indicated by the leading
+   these macros in your programs: As indicated by the leading
    underscore they are subject to change without notice. */
 #ifdef __GNUC__