Add timemsg() for logging
authorNeutron Soutmun <neo.neutron@gmail.com>
Tue, 19 Aug 2008 17:08:07 +0000 (00:08 +0700)
committerNeutron Soutmun <neo.neutron@gmail.com>
Sun, 2 Nov 2008 19:43:10 +0000 (02:43 +0700)
2008-08-20  Neutron Soutmun <neo.neutron@gmail.com>

* src/rahunas.h:
  - Add timemsg(), static funtion returning the time message for logging.
  - Adjust DP() for the DEBUG logging, which activate by define RH_DEBUG in
    compilation time.

* src/rahunas.c:
  - Adjust logmsg() to use the new timemsg().

ChangeLog
src/rahunasd.c
src/rahunasd.h

index 9abca68..512f4f7 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2008-08-20  Neutron Soutmun <neo.neutron@gmail.com>
+
+       * src/rahunas.h:
+         - Add timemsg(), static funtion returning the time message for logging. 
+         - Adjust DP() for the DEBUG logging, which activate by define RH_DEBUG in 
+           compilation time.
+
+       * src/rahunas.c:
+         - Adjust logmsg() to use the new timemsg().
+
 2008-08-19  Neutron Soutmun <neo.neutron@gmail.com>
 
        * src/rahunas.{h,c}:
index 537fa94..787f98a 100644 (file)
@@ -114,13 +114,10 @@ int rh_closelog(int fd)
 int logmsg(int priority, const char *msg, ...) 
 {
   int n, size = 256;
-  char time_msg[20];
-  time_t t;
        va_list ap;
        char *time_fmt = "%b %e %T";
        char *p = NULL;
   char *np = NULL;
-  struct tm *ltime = NULL;
 
        if (priority < RH_LOG_LEVEL)
          return 0;
@@ -154,20 +151,7 @@ int logmsg(int priority, const char *msg, ...)
        if (!p)
          return (-1);
 
-       t = time(NULL);
-       ltime = localtime(&t);
-       if (ltime == NULL) {
-               return (-1);
-       }
-
-       if ((np = rh_malloc(size+20)) == NULL) {
-               rh_free(p);
-               return (-1);
-       }
-
-       strftime(time_msg, sizeof time_msg, time_fmt, ltime);
-
-       fprintf(stderr, "%s : %s\n", time_msg, p);
+       fprintf(stderr, "%s : %s\n", timemsg(), p);
 
        rh_free(&p);
        rh_free(&np);
index a6b5f01..63f84a0 100644 (file)
 #define XMLSERVICE_PORT        8888
 #define XMLSERVICE_URL "/xmlrpc_service.php"
 
+
+struct rahunas_map {
+  struct rahunas_member *members;
+       in_addr_t first_ip;
+       in_addr_t last_ip;
+       unsigned int size;
+};
+
+struct rahunas_member {
+  unsigned short flags; 
+  unsigned short expired;
+       time_t session_start;
+       char *username;
+  char *session_id;
+};
+
+uint32_t iptoid(struct rahunas_map *map, const char *ip);
+char *idtoip(struct rahunas_map *map, uint32_t id);
+
+void *rh_malloc(size_t size);
+void rh_free(void **data);
+
+static char *timemsg()
+{
+  static char tmsg[32] = "";
+  char tfmt[] = "%b %e %T";
+  time_t t;
+
+  t = time(NULL);
+  strftime(tmsg, sizeof tmsg, tfmt, localtime(&t));
+  return tmsg; 
+}
+
 enum RH_LOG {
   RH_LOG_DEBUG,
        RH_LOG_NORMAL,
@@ -45,36 +78,13 @@ enum RH_LOG {
 #endif
 
 #ifdef RH_DEBUG
-#define DP(priority, format, args...) \
+#define DP(format, args...) \
   do {  \
-    if (priority > RH_LOG_LEVEL) { \
-      fprintf(stderr, "%s: %s (DBG): ", __FILE__, __FUNCTION__); \
-      fprintf(stderr, format "\n", ## args);
-    } \
+    fprintf(stderr, "%s - %s: %s (DBG): ", timemsg(), __FILE__, __FUNCTION__); \
+    fprintf(stderr, format "\n", ## args); \
   } while (0)
 #else
 #define DP(priority, format, args...)
 #endif
 
-struct rahunas_map {
-  struct rahunas_member *members;
-       in_addr_t first_ip;
-       in_addr_t last_ip;
-       unsigned int size;
-};
-
-struct rahunas_member {
-  unsigned short flags; 
-  unsigned short expired;
-       time_t session_start;
-       char *username;
-  char *session_id;
-};
-
-uint32_t iptoid(struct rahunas_map *map, const char *ip);
-char *idtoip(struct rahunas_map *map, uint32_t id);
-
-void *rh_malloc(size_t size);
-void rh_free(void **data);
-
 #endif // __RAHUNASD_H