[dpdk-dev] [PATCH] ethdev: fix missing parenthesis
Thomas Monjalon
thomas.monjalon at 6wind.com
Fri Jan 16 18:04:12 CET 2015
2015-01-16 09:40, Wodkowski, PawelX:
> > Good catch!
> > Was introduced in commit 4bdefaade6d1 (VMDQ enhancements).
> > Note that quite often, when a patch contains too much things,
> > we miss this kind of bugs. That's a reason to well split patches.
>
> What is most surprising I did not spotted this, neither the compiler
> what I think it should. It was my IDE which pointed possible bug.
>
> Are we using '-Wparentheses' switch? If not, we should consider
> to use this switch?
It's already included in -Wall which is set in WERROR_FLAGS.
--
Thomas
More information about the dev
mailing list