Changeset 1065 in MondoRescue for branches/stable/mondo/src/lib/mr_msg.c

Timestamp:

Jan 23, 2007, 3:48:48 AM (17 years ago)

Author:

Bruno Cornec

Message:

More tests in
mr_msg function doesn't cascade to _mr_msg anymore, as it doesn't seem to work correctly with multiple varargs entries (I've not found how to do it correctly, so I give up for the moment)
lib code pass valgrind without issue.

File:

• branches/stable/mondo/src/lib/mr_msg.c (modified) (4 diffs)

branches/stable/mondo/src/lib/mr_msg.c

@@ -20 +20 @@
 static char *mr_logfile = NULL;
 
+/*
+ * This function is in the lowest part of the tree
+ * It should not depend on any other function of the mr lib 
+ */
+
 /* Cleanup function for messages */
 void mr_msg_close(void) {
     free(mr_logfile);
     mr_logfile = NULL;
+    mr_loglevel = 0;
 }
 
@@ -47 +53 @@
  * but through other functions
  */
-void _mr_msg(int debug, const char *file, const char *function, int line, const char *fmt, ...) {
+void mr_msg(int debug, const char *fmt, ...) {
 
     int i = 0;
@@ -63 +69 @@
             return;
         }
-        va_start(args,fmt);
 
         // add 2 spaces to distinguish log levels
@@ -69 +74 @@
             for (i = 1; i < debug; i++)
                 fprintf(fout, "  ");
-            fprintf(fout, "%s->%s#%d: ", file, function, line);
+            fprintf(fout, "%s->%s#%d: ", __FILE__, __FUNCTION__, __LINE__);
         }
+        va_start(args,fmt);
         if (vfprintf(fout, fmt, args) < 0) {
             fprintf(stderr,"Unable to print to %s\n",mr_logfile);
         }
+        va_end(args);
 
-        fprintf(fout, "\n");
         if ((res = fclose(fout)) != 0) {
             fprintf(stderr,"Unable to close %s\n",mr_logfile);
         }
-        va_end(args);
     }
 }
-
-void mr_msg(int level, const char *fmt, ...) {
-
-    va_list args;
-
-    va_start(args,fmt);
-    _mr_msg(level, __FILE__, __FUNCTION__, __LINE__, fmt, args);
-    va_end(args);
-}