[dvblast-devel] syslog: don't print redundant info

[Christophe Massiot]

dvblast | branch: master | Christophe Massiot <cmassiot at openheadend.tv> | Fri Jul 17 18:15:49 2015 +0200| [5f7072f8d212d353a9d4ecf8291de13b5ba744f7] | committer: Christophe Massiot

syslog: don't print redundant info

> http://git.videolan.org/gitweb.cgi/dvblast.git/?a=commit;h=5f7072f8d212d353a9d4ecf8291de13b5ba744f7
---

 util.c |   99 +++++++++++++++++++++++++++++++++++++---------------------------
 1 file changed, 58 insertions(+), 41 deletions(-)

diff --git a/util.c b/util.c
index 55a99d8..8da0db5 100644
--- a/util.c
+++ b/util.c
@@ -1,7 +1,7 @@
 /*****************************************************************************
  * util.c
  *****************************************************************************
- * Copyright (C) 2004 VideoLAN
+ * Copyright (C) 2004, 2015 VideoLAN
  *
  * Authors: Christophe Massiot <massiot at via.ecp.fr>
  *
@@ -72,19 +72,24 @@ void msg_Disconnect( void )
  *****************************************************************************/
 void msg_Info( void *_unused, const char *psz_format, ... )
 {
-    if ( i_verbose >= VERB_INFO )
+    if ( i_verbose < VERB_INFO )
+        return;
+
+    va_list args;
+    va_start( args, psz_format );
+
+    if ( !i_syslog )
     {
-        va_list args;
         char psz_fmt[MAX_MSG];
-        va_start( args, psz_format );
-
         snprintf( psz_fmt, MAX_MSG, "info: %s\n", psz_format );
-        if ( i_syslog )
-            vsyslog( LOG_INFO, psz_fmt, args );
-        else
-            vfprintf( stderr, psz_fmt, args );
-        va_end(args);
+        vfprintf( stderr, psz_fmt, args );
     }
+    else
+    {
+        vsyslog( LOG_INFO, psz_format, args );
+    }
+
+    va_end(args);
 }
 
 /*****************************************************************************
@@ -92,19 +97,24 @@ void msg_Info( void *_unused, const char *psz_format, ... )
  *****************************************************************************/
 void msg_Err( void *_unused, const char *psz_format, ... )
 {
-    if ( i_verbose >= VERB_ERR )
+    if ( i_verbose < VERB_ERR )
+        return;
+
+    va_list args;
+    va_start( args, psz_format );
+
+    if ( !i_syslog )
     {
-        va_list args;
         char psz_fmt[MAX_MSG];
-        va_start( args, psz_format );
-
         snprintf( psz_fmt, MAX_MSG, "error: %s\n", psz_format );
-        if ( i_syslog )
-            vsyslog( LOG_ERR, psz_fmt, args );
-        else
-            vfprintf( stderr, psz_fmt, args );
-        va_end(args);
+        vfprintf( stderr, psz_fmt, args );
+    }
+    else
+    {
+        vsyslog( LOG_ERR, psz_format, args );
     }
+
+    va_end(args);
 }
 
 /*****************************************************************************
@@ -112,19 +122,24 @@ void msg_Err( void *_unused, const char *psz_format, ... )
  *****************************************************************************/
 void msg_Warn( void *_unused, const char *psz_format, ... )
 {
-    if ( i_verbose >= VERB_WARN )
+    if ( i_verbose < VERB_WARN )
+        return;
+
+    va_list args;
+    va_start( args, psz_format );
+
+    if ( !i_syslog )
     {
-        va_list args;
         char psz_fmt[MAX_MSG];
-        va_start( args, psz_format );
-
         snprintf( psz_fmt, MAX_MSG, "warning: %s\n", psz_format );
-        if ( i_syslog )
-            vsyslog( LOG_WARNING, psz_fmt, args );
-        else
-            vfprintf( stderr, psz_fmt, args );
-        va_end(args);
+        vfprintf( stderr, psz_fmt, args );
+    }
+    else
+    {
+        vsyslog( LOG_WARNING, psz_format, args );
     }
+
+    va_end(args);
 }
 
 /*****************************************************************************
@@ -132,19 +147,24 @@ void msg_Warn( void *_unused, const char *psz_format, ... )
  *****************************************************************************/
 void msg_Dbg( void *_unused, const char *psz_format, ... )
 {
-    if ( i_verbose >= VERB_DBG )
+    if ( i_verbose < VERB_DBG )
+        return;
+
+    va_list args;
+    va_start( args, psz_format );
+
+    if ( !i_syslog )
     {
-        va_list args;
         char psz_fmt[MAX_MSG];
-        va_start( args, psz_format );
-
         snprintf( psz_fmt, MAX_MSG, "debug: %s\n", psz_format );
-        if ( i_syslog )
-            vsyslog( LOG_DEBUG, psz_fmt, args );
-        else
-            vfprintf( stderr, psz_fmt, args );
-        va_end(args);
+        vfprintf( stderr, psz_fmt, args );
+    }
+    else
+    {
+        vsyslog( LOG_DEBUG, psz_format, args );
     }
+
+    va_end(args);
 }
 
 /*****************************************************************************
@@ -157,10 +177,7 @@ void msg_Raw( void *_unused, const char *psz_format, ... )
     va_start( args, psz_format );
 
     snprintf( psz_fmt, MAX_MSG, "%s\n", psz_format );
-    if ( i_syslog )
-        vsyslog( LOG_NOTICE, psz_fmt, args );
-    else
-        vfprintf( stdout, psz_fmt, args );
+    vfprintf( stderr, psz_fmt, args );
     va_end(args);
 }