From fca338a06c54f272f05aa12e362279e113130284 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Miloslav=20Trma=C4=8D?= Date: Thu, 20 Sep 2012 20:24:00 +0200 Subject: [PATCH] Drop unused return value of virLogOutputFunc MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Nothing uses the return value, and creating it requries otherwise unnecessary strlen () calls. This cleanup is conceptually independent from the rest of the series (although the later patches won't apply without it). This just seems a good opportunity to clean this up, instead of entrenching the unnecessary return value in the virLogOutputFunc instance that will be added in this series. Signed-off-by: Miloslav Trmač --- src/util/logging.c | 56 +++++++++++++++++++++------------------------- src/util/logging.h | 14 +++++------- tests/testutils.c | 5 ++--- 3 files changed, 34 insertions(+), 41 deletions(-) diff --git a/src/util/logging.c b/src/util/logging.c index bc76a2dc2c..cdaf86ebfc 100644 --- a/src/util/logging.c +++ b/src/util/logging.c @@ -98,12 +98,12 @@ static virLogPriority virLogDefaultPriority = VIR_LOG_DEFAULT; static int virLogResetFilters(void); static int virLogResetOutputs(void); -static int virLogOutputToFd(const char *category, int priority, - const char *funcname, long long linenr, - const char *timestamp, - unsigned int flags, - const char *str, - void *data); +static void virLogOutputToFd(const char *category, int priority, + const char *funcname, long long linenr, + const char *timestamp, + unsigned int flags, + const char *str, + void *data); /* * Logs accesses must be serialized though a mutex @@ -807,32 +807,29 @@ static void virLogStackTraceToFd(int fd) #undef STRIP_DEPTH } -static int virLogOutputToFd(const char *category ATTRIBUTE_UNUSED, - int priority ATTRIBUTE_UNUSED, - const char *funcname ATTRIBUTE_UNUSED, - long long linenr ATTRIBUTE_UNUSED, - const char *timestamp, - unsigned int flags, - const char *str, - void *data) +static void virLogOutputToFd(const char *category ATTRIBUTE_UNUSED, + int priority ATTRIBUTE_UNUSED, + const char *funcname ATTRIBUTE_UNUSED, + long long linenr ATTRIBUTE_UNUSED, + const char *timestamp, + unsigned int flags, + const char *str, + void *data) { int fd = (intptr_t) data; - int ret; char *msg; if (fd < 0) - return -1; + return; if (virAsprintf(&msg, "%s: %s", timestamp, str) < 0) - return -1; + return; - ret = safewrite(fd, msg, strlen(msg)); + ignore_value(safewrite(fd, msg, strlen(msg))); VIR_FREE(msg); if (flags & VIR_LOG_STACK_TRACE) virLogStackTraceToFd(fd); - - return ret; } static void virLogCloseFd(void *data) @@ -865,18 +862,18 @@ static int virLogAddOutputToFile(int priority, const char *file) { } #if HAVE_SYSLOG_H -static int virLogOutputToSyslog(const char *category ATTRIBUTE_UNUSED, - int priority, - const char *funcname ATTRIBUTE_UNUSED, - long long linenr ATTRIBUTE_UNUSED, - const char *timestamp ATTRIBUTE_UNUSED, - unsigned int flags, - const char *str, - void *data ATTRIBUTE_UNUSED) +static void virLogOutputToSyslog(const char *category ATTRIBUTE_UNUSED, + int priority, + const char *funcname ATTRIBUTE_UNUSED, + long long linenr ATTRIBUTE_UNUSED, + const char *timestamp ATTRIBUTE_UNUSED, + unsigned int flags, + const char *str, + void *data ATTRIBUTE_UNUSED) { int prio; - virCheckFlags(VIR_LOG_STACK_TRACE, -1); + virCheckFlags(VIR_LOG_STACK_TRACE,); switch (priority) { case VIR_LOG_DEBUG: @@ -895,7 +892,6 @@ static int virLogOutputToSyslog(const char *category ATTRIBUTE_UNUSED, prio = LOG_ERR; } syslog(prio, "%s", str); - return strlen(str); } static char *current_ident = NULL; diff --git a/src/util/logging.h b/src/util/logging.h index 04a613e23b..256e81699b 100644 --- a/src/util/logging.h +++ b/src/util/logging.h @@ -94,15 +94,13 @@ typedef enum { * @data: extra output logging data * * Callback function used to output messages - * - * Returns the number of bytes written or -1 in case of error */ -typedef int (*virLogOutputFunc) (const char *category, int priority, - const char *funcname, long long linenr, - const char *timestamp, - unsigned int flags, - const char *str, - void *data); +typedef void (*virLogOutputFunc) (const char *category, int priority, + const char *funcname, long long linenr, + const char *timestamp, + unsigned int flags, + const char *str, + void *data); /** * virLogCloseFunc: diff --git a/tests/testutils.c b/tests/testutils.c index 34edde9773..b555e06214 100644 --- a/tests/testutils.c +++ b/tests/testutils.c @@ -478,7 +478,7 @@ struct virtTestLogData { static struct virtTestLogData testLog = { VIR_BUFFER_INITIALIZER }; -static int +static void virtTestLogOutput(const char *category ATTRIBUTE_UNUSED, int priority ATTRIBUTE_UNUSED, const char *funcname ATTRIBUTE_UNUSED, @@ -489,9 +489,8 @@ virtTestLogOutput(const char *category ATTRIBUTE_UNUSED, void *data) { struct virtTestLogData *log = data; - virCheckFlags(VIR_LOG_STACK_TRACE, -1); + virCheckFlags(VIR_LOG_STACK_TRACE,); virBufferAsprintf(&log->buf, "%s: %s", timestamp, str); - return strlen(timestamp) + 2 + strlen(str); } static void -- GitLab