From mboxrd@z Thu Jan 1 00:00:00 1970 From: jeremie.galarneau@efficios.com (=?UTF-8?Q?J=C3=A9r=C3=A9mie_Galarneau?=) Date: Thu, 19 May 2016 00:13:15 -0400 Subject: [lttng-dev] [PATCH lttng-tools 06/10] Fix: error.h: add missing parenthesis around macro parameter In-Reply-To: <1463594659-10964-7-git-send-email-mathieu.desnoyers@efficios.com> References: <1463594659-10964-1-git-send-email-mathieu.desnoyers@efficios.com> <1463594659-10964-7-git-send-email-mathieu.desnoyers@efficios.com> Message-ID: Merged, thanks! Jérémie On Wed, May 18, 2016 at 2:04 PM, Mathieu Desnoyers wrote: > Signed-off-by: Mathieu Desnoyers > --- > src/common/error.h | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/src/common/error.h b/src/common/error.h > index 3ab1d46..ca5af86 100644 > --- a/src/common/error.h > +++ b/src/common/error.h > @@ -75,18 +75,18 @@ extern int lttng_opt_mi; > */ > #define __lttng_print(type, fmt, args...) \ > do { \ > - if (!lttng_opt_quiet&& !lttng_opt_mi && \ > - type == PRINT_MSG) { \ > + if (!lttng_opt_quiet && !lttng_opt_mi && \ > + (type) == PRINT_MSG) { \ > fprintf(stdout, fmt, ## args); \ > } else if (!lttng_opt_quiet && !lttng_opt_mi && \ > - (((type & PRINT_DBG) && lttng_opt_verbose == 1) || \ > - ((type & (PRINT_DBG | PRINT_DBG2)) && \ > + ((((type) & PRINT_DBG) && lttng_opt_verbose == 1) || \ > + (((type) & (PRINT_DBG | PRINT_DBG2)) && \ > lttng_opt_verbose == 2) || \ > - ((type & (PRINT_DBG | PRINT_DBG2 | PRINT_DBG3)) && \ > + (((type) & (PRINT_DBG | PRINT_DBG2 | PRINT_DBG3)) && \ > lttng_opt_verbose == 3))) { \ > fprintf(stderr, fmt, ## args); \ > } else if (!lttng_opt_quiet && \ > - (type & (PRINT_WARN | PRINT_ERR | PRINT_BUG))) { \ > + ((type) & (PRINT_WARN | PRINT_ERR | PRINT_BUG))) { \ > fprintf(stderr, fmt, ## args); \ > } \ > } while (0); > -- > 2.1.4 > -- Jérémie Galarneau EfficiOS Inc. http://www.efficios.com