[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