Stephen Hemminger <step...@networkplumber.org> writes:
> On Thu, 7 Jan 2021 09:13:34 +0200 > Roi Dayan <r...@nvidia.com> wrote: > >> +#define _IS_JSON_CONTEXT(type) ((type & PRINT_JSON || type & PRINT_ANY) && >> is_json_context()) >> +#define _IS_FP_CONTEXT(type) (!is_json_context() && (type & PRINT_FP || >> type & PRINT_ANY)) > > You could fold the comparisons? and why are the two options doing comparison > in different order? > > #define _IS_JSON_CONTEXT(type) (is_json_context() && (type & (PRINT_JSON | > PRINT_ANY)) > #define _IS_FP_CONTEXT(type) (!is_json_context() && (type & (PRINT_FP || > PRINT_ANY)) (s/||/|/)
