lib: Change event_set_forced_debug(e, FALSE) to be a no-op

diff --git a/src/lib/event-log.c b/src/lib/event-log.c
index 2e3f72e027cda0f28a4e18cd254bf21b1fdd47c2..ee9aa4827387476b0cd3b4422629119efb20725e 100644 (file)
--- a/src/lib/event-log.c
+++ b/src/lib/event-log.c
@@ -202,7 +202,8 @@ void event_logv(struct event *event, const struct event_log_params *params,
 
 struct event *event_set_forced_debug(struct event *event, bool force)
 {
-       event->forced_debug = force;
+       if (force)
+               event->forced_debug = TRUE;
        return event;
 }
 

diff --git a/src/lib/event-log.h b/src/lib/event-log.h
index 77f61805f0e2e0ba0a1b764c5f161b47ee9e28bd..b11999089bbabcd73212a82bfc84f5b8fe1ab5b0 100644 (file)
--- a/src/lib/event-log.h
+++ b/src/lib/event-log.h
@@ -58,12 +58,8 @@ void event_logv(struct event *event, const struct event_log_params *params,
    afterwards. It doesn't apply to existing child events (mainly for
    performance reasons).
 
-   Note that it's always recommended to use e.g.:
-     if (set->debug) event_set_forced_debug(event, TRUE); // good
-   instead of
-     event_set_forced_debug(event, set->debug); // bad
-   This is because the event may already have had debugging enabled via the
-   parent event. Forcing it to FALSE is most likely not wanted. */
+   Note that event_set_forced_debug(event, FALSE) is a no-op. To disable
+   forced-debug, use event_unset_forced_debug(event). */
 struct event *event_set_forced_debug(struct event *event, bool force);
 /* Set the forced-debug to FALSE */
 struct event *event_unset_forced_debug(struct event *event);