[ovs-dev] [PATCH v2 05/11] dpif-netdev: refactor to multiple header files

0-day Robot robot at bytheb.org
Fri Oct 30 20:13:17 UTC 2020


Bleep bloop.  Greetings Harry van Haaren, I am a robot and I have tried out your patch.
Thanks for your contribution.

I encountered some error that I wasn't expecting.  See the details below.


checkpatch:
ERROR: Improper whitespace around control block
#215 FILE: lib/dpif-netdev-private-dfc.h:111:
#define EMC_FOR_EACH_POS_WITH_HASH(EMC, CURRENT_ENTRY, HASH)                 \

ERROR: Improper whitespace around control block
#331 FILE: lib/dpif-netdev-private-dfc.h:227:
    EMC_FOR_EACH_POS_WITH_HASH(cache, current_entry, key->hash) {

ERROR: Improper whitespace around control block
#449 FILE: lib/dpif-netdev-private-dpcls.h:95:
#define NETDEV_FLOW_KEY_FOR_EACH_IN_FLOWMAP(VALUE, KEY, FLOWMAP)   \

ERROR: Inappropriate bracing around statement
#450 FILE: lib/dpif-netdev-private-dpcls.h:96:
    MINIFLOW_FOR_EACH_IN_FLOWMAP (VALUE, &(KEY)->mf, FLOWMAP)

WARNING: Line has trailing whitespace
#994 FILE: lib/dpif-netdev-private.h:48:
 *  

Lines checked: 1549, Warnings: 1, Errors: 4


Please check this out.  If you feel there has been an error, please email aconole at redhat.com

Thanks,
0-day Robot


More information about the dev mailing list