]> sigrok.org Git - libsigrok.git/commitdiff
log: Move log level check so that it affects all handlers.
authorUwe Hermann <redacted>
Sat, 21 Apr 2018 22:23:10 +0000 (00:23 +0200)
committerUwe Hermann <redacted>
Sat, 21 Apr 2018 22:23:10 +0000 (00:23 +0200)
Before this change, the loglevel check would only be performed for the
default log handler in libsigrok, but not for other handlers set
via sr_log_callback_set().

This fixes bug #698.

src/log.c

index 55c5270f93a42db3a602191f41cb401f15508393..caffa2a2d4900f8532b82455288a748aea6b4ba7 100644 (file)
--- a/src/log.c
+++ b/src/log.c
@@ -170,9 +170,7 @@ static int sr_logv(void *cb_data, int loglevel, const char *format, va_list args
        /* This specific log callback doesn't need the void pointer data. */
        (void)cb_data;
 
-       /* Only output messages of at least the selected loglevel(s). */
-       if (loglevel > cur_loglevel)
-               return SR_OK;
+       (void)loglevel;
 
        if (cur_loglevel >= LOGLEVEL_TIMESTAMP) {
                elapsed_us = g_get_monotonic_time() - sr_log_start_time;
@@ -217,6 +215,10 @@ SR_PRIV int sr_log(int loglevel, const char *format, ...)
        int ret;
        va_list args;
 
+       /* Only output messages of at least the selected loglevel(s). */
+       if (loglevel > cur_loglevel)
+               return SR_OK;
+
        va_start(args, format);
        ret = sr_log_cb(sr_log_cb_data, loglevel, format, args);
        va_end(args);