pud: simplify pudError a bit
authorFerry Huberts <ferry.huberts@pelagic.nl>
Mon, 16 Jul 2012 09:43:11 +0000 (11:43 +0200)
committerFerry Huberts <ferry.huberts@pelagic.nl>
Mon, 16 Jul 2012 10:32:43 +0000 (12:32 +0200)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
lib/pud/src/pud.c

index 69c63cf..dfa6940 100644 (file)
@@ -43,17 +43,16 @@ static DeDupList deDupList;
  */
 void pudError(bool useErrno, const char *format, ...) {
        char strDesc[256];
-       char *stringErr = NULL;
-
-       if (useErrno) {
-               stringErr = strerror(errno);
-       }
+       const char *colon;
+       const char *stringErr;
 
        if ((format == NULL) || (*format == '\0')) {
-               if (useErrno) {
-                       olsr_printf(0, "%s: %s\n", PUD_PLUGIN_ABBR, stringErr);
+               strDesc[0] = '\0';
+               colon = "";
+               if (!useErrno) {
+                       stringErr = "Unknown error";
                } else {
-                       olsr_printf(0, "%s: Unknown error\n", PUD_PLUGIN_ABBR);
+                       stringErr = strerror(errno);
                }
        } else {
                va_list arglist;
@@ -62,14 +61,15 @@ void pudError(bool useErrno, const char *format, ...) {
                vsnprintf(strDesc, sizeof(strDesc), format, arglist);
                va_end(arglist);
 
-               strDesc[sizeof(strDesc) - 1] = '\0'; /* Ensures null termination */
-
                if (useErrno) {
-                       olsr_printf(0, "%s: %s: %s\n", PUD_PLUGIN_ABBR, strDesc, stringErr);
+                       colon = ": ";
+                       stringErr = strerror(errno);
                } else {
-                       olsr_printf(0, "%s: %s\n", PUD_PLUGIN_ABBR, strDesc);
+                       colon = "";
+                       stringErr = "";
                }
        }
+       olsr_printf(0, "%s: %s%s%s\n", PUD_PLUGIN_ABBR, strDesc, colon, stringErr);
 }
 
 /**