From: niki <niki>
Date: Sat, 17 Dec 2005 02:35:20 +0000 (+0000)
Subject: fixed static filename bug, cleanup of --disable-debug mode
X-Git-Tag: collectd-3.6.0~49
X-Git-Url: https://git.verplant.org/?a=commitdiff_plain;h=7b335cff468857d2ce5c4a0ad650f5412c391359;p=collectd.git

fixed static filename bug, cleanup of --disable-debug mode
---

diff --git a/src/utils_debug.c b/src/utils_debug.c
index db59525e..9a6c1c5e 100644
--- a/src/utils_debug.c
+++ b/src/utils_debug.c
@@ -23,16 +23,13 @@
 #include "common.h"
 #include "utils_debug.h"
 
-/* *** *** ***   global variables   *** *** *** */
 #if COLLECT_DEBUG
 
-#define DEFAULT_FILENAME "collectd.log"
-
 static void cu_vdebug(const char *file, int line, const char *func,
 	const char *format, va_list ap);
 
 /* if preemptive threads are used, these vars need some sort of locking! */
-/* pth is non-preemptive, so no locking is necessary (?) */
+/* pth is non-preemptive, so no locking is necessary */
 static FILE *cu_debug_file = NULL;
 static char *cu_debug_filename = NULL;
 
@@ -71,7 +68,7 @@ cu_debug(const char *file, int line, const char *func,
 
 int
 cu_debug_startfile(const char *file, int line, const char *func,
-	const char *format, ...)
+	const char *filename, const char *format, ...)
 {
 	va_list ap;
 
@@ -82,7 +79,7 @@ cu_debug_startfile(const char *file, int line, const char *func,
 	}
 
 	if(cu_debug_filename == NULL) {
-		cu_debug_filename = sstrdup(DEFAULT_FILENAME);
+		cu_debug_filename = sstrdup(filename);
 	}
 
 	cu_debug_file = fopen(cu_debug_filename, "a");
@@ -169,27 +166,5 @@ cu_debug_resetfile(const char *file, int line, const char *func,
 } /* int cu_debug_resetfile(const char *file, int line, const char *func,
         const char *filename) */
 
-#else /* !COLLECT_DEBUG */
-
-void
-cu_debug(const char *file, int line, const char *func, const char *format, ...)
-{
-}
-int
-cu_debug_startfile(const char *file, int line, const char *func, const char *format, ...)
-{
-	return EXIT_SUCCESS;
-}
-int
-cu_debug_stopfile(const char *file, int line, const char *func, const char *format, ...)
-{
-	return EXIT_SUCCESS;
-}
-int
-cu_debug_resetfile(const char *file, int line, const char *func, const char *filename)
-{
-	return EXIT_SUCCESS;
-}
-
 #endif /* COLLECT_DEBUG */
 
diff --git a/src/utils_debug.h b/src/utils_debug.h
index 247b141d..561e6b1e 100644
--- a/src/utils_debug.h
+++ b/src/utils_debug.h
@@ -21,14 +21,19 @@
  *   Niki W. Waibel <niki.waibel@gmx.net>
 **/
 
+/* BUG: at the moment you can use DBG_STARTFILE only ONCE in
+   your program. more then one logfile is not supported. */
+
 #if !COLLECTD_UTILS_DEBUG_H
 #define COLLECTD_UTILS_DEBUG_H 1
 
+#if COLLECT_DEBUG
+
 #define DBG(...) cu_debug(__FILE__, __LINE__, __func__, \
 	__VA_ARGS__)
 
-#define DBG_STARTFILE(...) cu_debug_startfile(__FILE__, __LINE__, \
-	__func__, __VA_ARGS__)
+#define DBG_STARTFILE(file, ...) cu_debug_startfile(__FILE__, __LINE__, \
+	__func__, file, __VA_ARGS__)
 #define DBG_STOPFILE(...) cu_debug_stopfile(__FILE__, __LINE__, \
 	__func__, __VA_ARGS__)
 
@@ -39,12 +44,21 @@ void cu_debug(const char *file, int line, const char *func,
 	const char *format, ...);
 
 int cu_debug_startfile(const char *file, int line, const char *func,
-	const char *format, ...);
+	const char *filename, const char *format, ...);
 int cu_debug_stopfile(const char *file, int line, const char *func,
 	const char *format, ...);
 
 int cu_debug_resetfile(const char *file, int line, const char *func,
 	const char *filename);
 
+#else /* !COLLECT_DEBUG */
+
+#define DBG(...)                   /* DBG(...) */
+#define DBG_STARTFILE(file, ...)   /* DBG_STARTFILE(file, ...) */
+#define DBG_STOPFILE(...)          /* DBG_STOPFILE(...) */
+#define DBG_RESETFILE(file)        /* DBG_RESETFILE(file) */
+
+#endif /* COLLECT_DEBUG */
+
 #endif /* !COLLECTD_UTILS_DEBUG_H */