Streamline DEBUG_ARRAY, DEBUG_BUFFER, DEBUG_IO, DEBUG_ZIP

Change all #define's to follow the form
	#define DEBUG_xxx {0|1}
to disable (0, default) or enable (1) additional debug messages.

And somewhat enhance some DEBUG_BUFFER messages.

diff --git a/src/ngircd/array.c b/src/ngircd/array.c
index af66edd9305580aea5b53471a00a1300bfe0aaac..4cc793f18c02b3f4bea6dd1f46ed2914441740a5 100644 (file)
--- a/src/ngircd/array.c
+++ b/src/ngircd/array.c
@@ -14,6 +14,9 @@
  * Functions to dynamically allocate arrays.
  */
 
+/* Additionan debug messages related to array handling: 0=off / 1=on */
+#define DEBUG_ARRAY 0
+
 #include "array.h"
 
 #include <assert.h>
@@ -21,13 +24,10 @@
 #include <string.h>
 #include <unistd.h>
 
-#ifdef DEBUG_ARRAY
+#if DEBUG_ARRAY
 # include "log.h"
 #endif
 
-/* Enable more Debug messages in alloc / append / memmove code. */
-/* #define DEBUG_ARRAY */
-
 #define array_UNUSABLE(x)      ( !(x)->mem )
 
 
@@ -67,7 +67,7 @@ array_alloc(array * a, size_t size, size_t pos)
                return NULL;
 
        if (a->allocated < alloc) {
-#ifdef DEBUG_ARRAY
+#if DEBUG_ARRAY
                Log(LOG_DEBUG, "array_alloc(): changing size from %u to %u bytes.",
                    a->allocated, alloc);
 #endif
@@ -168,7 +168,7 @@ array_catb(array * dest, const char *src, size_t len)
 
        assert(ptr != NULL);
 
-#ifdef DEBUG_ARRAY
+#if DEBUG_ARRAY
        Log(LOG_DEBUG,
            "array_catb(): appending %u bytes to array (now %u bytes in array).",
            len, tmp);
@@ -248,7 +248,7 @@ void
 array_free(array * a)
 {
        assert(a != NULL);
-#ifdef DEBUG_ARRAY
+#if DEBUG_ARRAY
        Log(LOG_DEBUG,
            "array_free(): %u bytes free'd (%u bytes still used at time of free()).",
            a->allocated, a->used);
@@ -314,7 +314,7 @@ array_moveleft(array * a, size_t membersize, size_t pos)
        if (!bytepos)
                return; /* nothing to do */
 
-#ifdef DEBUG_ARRAY
+#if DEBUG_ARRAY
        Log(LOG_DEBUG,
            "array_moveleft(): %u bytes used in array, starting at position %u.",
            a->used, bytepos);

diff --git a/src/ngircd/conn-zip.c b/src/ngircd/conn-zip.c
index 5ebb2a04189035a3f814e74fdbeda0a5abffb9a0..e21dd345777a35c599f308aee587c6dee377c4dd 100644 (file)
--- a/src/ngircd/conn-zip.c
+++ b/src/ngircd/conn-zip.c
@@ -9,6 +9,8 @@
  * Please read the file COPYING, README and AUTHORS for more information.
  */
 
+#define CONN_MODULE
+
 #include "portab.h"
 
 /**
@@ -16,13 +18,11 @@
  * Connection compression using ZLIB
  */
 
-#define CONN_MODULE
+/* Additionan debug messages related to ZIP compression: 0=off / 1=on */
+#define DEBUG_ZIP 0
 
 #ifdef ZLIB
 
-/* enable more zlib related debug messages: */
-/* #define DEBUG_ZLIB */
-
 #include <assert.h>
 #include <string.h>
 #include <zlib.h>
@@ -141,7 +141,7 @@ Zip_Flush( CONN_ID Idx )
        out->next_out = zipbuf;
        out->avail_out = (uInt)sizeof zipbuf;
 
-#ifdef DEBUG_ZIP
+#if DEBUG_ZIP
        Log(LOG_DEBUG, "out->avail_in %d, out->avail_out %d",
                out->avail_in, out->avail_out);
 #endif
@@ -164,7 +164,7 @@ Zip_Flush( CONN_ID Idx )
        assert(out->avail_out <= WRITEBUFFER_SLINK_LEN);
 
        zipbuf_used = WRITEBUFFER_SLINK_LEN - out->avail_out;
-#ifdef DEBUG_ZIP
+#if DEBUG_ZIP
        Log(LOG_DEBUG, "zipbuf_used: %d", zipbuf_used);
 #endif
        if (!array_catb(&My_Connections[Idx].wbuf,
@@ -216,7 +216,7 @@ Unzip_Buffer( CONN_ID Idx )
        in->next_out = unzipbuf;
        in->avail_out = (uInt)sizeof unzipbuf;
 
-#ifdef DEBUG_ZIP
+#if DEBUG_ZIP
        Log(LOG_DEBUG, "in->avail_in %d, in->avail_out %d",
                in->avail_in, in->avail_out);
 #endif
@@ -231,7 +231,7 @@ Unzip_Buffer( CONN_ID Idx )
        assert(z_rdatalen >= in->avail_in);
        in_len = z_rdatalen - in->avail_in;
        unzipbuf_used = READBUFFER_LEN - in->avail_out;
-#ifdef DEBUG_ZIP
+#if DEBUG_ZIP
        Log(LOG_DEBUG, "unzipbuf_used: %d - %d = %d", READBUFFER_LEN,
                in->avail_out, unzipbuf_used);
 #endif

diff --git a/src/ngircd/conn.c b/src/ngircd/conn.c
index 4ed586206f42107b0e32132530631545fede4ea4..4dfe62fbdb85935030dc3112032baee6668ba9ff 100644 (file)
--- a/src/ngircd/conn.c
+++ b/src/ngircd/conn.c
@@ -9,8 +9,6 @@
  * Please read the file COPYING, README and AUTHORS for more information.
  */
 
-#undef DEBUG_BUFFER
-
 #define CONN_MODULE
 
 #include "portab.h"
@@ -20,6 +18,9 @@
  * Connection management
  */
 
+/* Additionan debug messages related to buffer handling: 0=off / 1=on */
+#define DEBUG_BUFFER 0
+
 #include <assert.h>
 #ifdef PROTOTYPES
 # include <stdarg.h>
@@ -1265,7 +1266,7 @@ Handle_Write( CONN_ID Idx )
                return true;
        }
 
-#ifdef DEBUG_BUFFER
+#if DEBUG_BUFFER
        LogDebug
            ("Handle_Write() called for connection %d, %ld bytes pending ...",
             Idx, wdatalen);
@@ -1798,10 +1799,6 @@ Handle_Buffer(CONN_ID Idx)
                        return 0; /* error -> connection has been closed */
 
                array_moveleft(&My_Connections[Idx].rbuf, 1, len);
-#ifdef DEBUG_BUFFER
-               LogDebug("Connection %d: %d bytes left in read buffer.",
-                        Idx, array_bytes(&My_Connections[Idx].rbuf));
-#endif
 #ifdef ZLIB
                if ((!old_z) && (My_Connections[Idx].options & CONN_ZIP) &&
                    (array_bytes(&My_Connections[Idx].rbuf) > 0)) {
@@ -1824,6 +1821,12 @@ Handle_Buffer(CONN_ID Idx)
                }
 #endif
        }
+#if DEBUG_BUFFER
+       LogDebug("Connection %d: Processed %ld commands (max=%ld), %ld bytes. %ld bytes left in read buffer.",
+                Idx, i, maxcmd, len_processed,
+                array_bytes(&My_Connections[Idx].rbuf));
+#endif
+
        return len_processed;
 } /* Handle_Buffer */
 

diff --git a/src/ngircd/io.c b/src/ngircd/io.c
index 779e7492ab010e8b9d81d3aa81f5b6b2a53e5c11..037c4afca2af7c0cf9662058ee2a18d3a92b12eb 100644 (file)
--- a/src/ngircd/io.c
+++ b/src/ngircd/io.c
@@ -17,6 +17,9 @@
  * I/O abstraction interface.
  */
 
+/* Extra debug messages in event add/delete/callback code: 0=off / 1=on */
+#define DEBUG_IO 0
+
 #include <assert.h>
 #include <string.h>
 #include <sys/types.h>
@@ -28,9 +31,6 @@
 #include "io.h"
 #include "log.h"
 
-/* Enables extra debug messages in event add/delete/callback code. */
-/* #define DEBUG_IO */
-
 typedef struct {
 #ifdef PROTOTYPES
  void (*callback)(int, short);
@@ -144,7 +144,7 @@ static array io_events;
 
 static void io_docallback PARAMS((int fd, short what));
 
-#ifdef DEBUG_IO
+#if DEBUG_IO
 static void
 io_debug(const char *s, int fd, int what)
 {