diag-macrocs: introduce pp_diag_allow, used in fsm.c

pp_diag_allow(ppi, thing, level) can be used as a conditional to mask off diagnostic prints that cannot be individually written as pp_diag() (this mainly applies to frame dumping, introduced in later commits). Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

diag-macrocs: introduce pp_diag_allow, used in fsm.c
pp_diag_allow(ppi, thing, level) can be used as a conditional to mask off diagnostic prints that cannot be individually written as pp_diag() (this mainly applies to frame dumping, introduced in later commits). Signed-off-by: Alessandro Rubini <rubini@gnudd.com>
5589514b · Alessandro Rubini · 07fc9b14 · 5589514b · 5589514b
Commit 5589514b authored Mar 14, 2013 by Alessandro Rubini
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 1 deletion

fsm.c fsm.c +1 -1

diag-macros.h include/ppsi/diag-macros.h +3 -0

No files found.
--- a/fsm.c
+++ b/fsm.c
@@ -16,7 +16,7 @@ static void pp_fsm_printf(struct pp_instance *ppi, char *fmt, ...)
 	TimeInternal t;
 	unsigned long oflags = pp_global_flags;

-	if (!__PP_DIAG_ALLOW(ppi, pp_dt_fsm, 1))
+	if (!pp_diag_allow(ppi, fsm, 1))
 		return;

 	/* temporarily set NOTIMELOG, as we'll print the time ourselves */

--- a/include/ppsi/diag-macros.h
+++ b/include/ppsi/diag-macros.h
@@ -128,6 +128,9 @@ extern void __pp_diag(struct pp_instance *ppi, enum pp_diag_things th,
 	PP_HAS_DIAG; /* return 1 if done, 0 if not done */		\
 	})

+#define pp_diag_allow(ppi_, th_, level_) \
+		(PP_HAS_DIAG && __PP_DIAG_ALLOW(ppi_,  pp_dt_ ## th_, level_))
+
 /*
 * And this is the parser of the string. Internally it obeys VERB_LOG_MESGS
 * to set the value to 0xfffffff0 to be compatible with previous usage.