[PATCH] more logging.h cleanups to be a bit more flexible.

diff --git a/logging.h b/logging.h
index df0d7dedcea1ceb1b72ccd9a74341de84815913e..2ba2ac49654c919ca0f4619943a86c186fc4394d 100644 (file)
--- a/logging.h
+++ b/logging.h
@@ -57,23 +57,11 @@
        } while (0)
 #endif
 
-/* each program must declare this variable and function somewhere */
-extern unsigned char logname[42];
-extern int log_ok(void);
-
-static void log_message (int level, const char *format, ...)
+extern void log_message (int level, const char *format, ...)
        __attribute__ ((format (printf, 2, 3)));
-static inline void log_message (int level, const char *format, ...)
-{
-       va_list args;
 
-       if (!log_ok())
-               return;
-
-       va_start(args, format);
-       vsyslog(level, format, args);
-       va_end(args);
-}
+/* each program that uses syslog must declare this variable somewhere */
+extern unsigned char logname[42];
 
 #undef init_logging
 static inline void init_logging(char *program_name)

diff --git a/udev.c b/udev.c
index 0f01fa41d2a04e6b08cda306c7d61d6345c146de..9f10cc212539466e5e03edc05016976d90069e24 100644 (file)
--- a/udev.c
+++ b/udev.c
@@ -38,12 +38,21 @@
 /* global variables */
 char **main_argv;
 char **main_envp;
-unsigned char logname[42];
 
-int log_ok(void)
+#ifdef LOG
+unsigned char logname[42];
+void log_message (int level, const char *format, ...)
 {
-       return udev_log;
+       va_list args;
+
+       if (!udev_log)
+               return;
+
+       va_start(args, format);
+       vsyslog(level, format, args);
+       va_end(args);
 }
+#endif
 
 static void sig_handler(int signum)
 {

diff --git a/udevd.c b/udevd.c
index 14616b9824e16dd509f74ce00c0996b2624c0119..9fbb4e964b6effeb7376cd134fba8180522f7bda 100644 (file)
--- a/udevd.c
+++ b/udevd.c
@@ -51,12 +51,17 @@ LIST_HEAD(running_list);
 static void exec_queue_manager(void);
 static void msg_queue_manager(void);
 
+#ifdef LOG
 unsigned char logname[42];
-
-int log_ok(void)
+void log_message (int level, const char *format, ...)
 {
-       return 1;
+       va_list args;
+
+       va_start(args, format);
+       vsyslog(level, format, args);
+       va_end(args);
 }
+#endif
 
 static void msg_dump_queue(void)
 {

diff --git a/udevinfo.c b/udevinfo.c
index 103ebce08cad295a2d7443237e37e545031705cf..44eb1427092abfda36f78407ffe84aa410609308 100644 (file)
--- a/udevinfo.c
+++ b/udevinfo.c
@@ -38,12 +38,18 @@
 
 char **main_argv;
 int main_argc;
-unsigned char logname[42];
 
-int log_ok(void)
+#ifdef LOG
+unsigned char logname[42];
+void log_message (int level, const char *format, ...)
 {
-       return 1;
+       va_list args;
+
+       va_start(args, format);
+       vsyslog(level, format, args);
+       va_end(args);
 }
+#endif
 
 static int print_all_attributes(const char *path)
 {

diff --git a/udevsend.c b/udevsend.c
index b2fbcbbca6f20139bbf6312703f34623fd7670cf..c738fe22459fc55e43211236e9125d3b18963b48 100644 (file)
--- a/udevsend.c
+++ b/udevsend.c
@@ -40,12 +40,17 @@
 #include "udevd.h"
 #include "logging.h"
 
+#ifdef LOG
 unsigned char logname[42];
-
-int log_ok(void)
+void log_message (int level, const char *format, ...)
 {
-       return 1;
+       va_list args;
+
+       va_start(args, format);
+       vsyslog(level, format, args);
+       va_end(args);
 }
+#endif
 
 static inline char *get_action(void)
 {