[ovs-dev] [PATCH] hmap.h: Fix Coverity false positive

0-day Robot robot at bytheb.org
Thu Mar 19 22:10:44 UTC 2020


Bleep bloop.  Greetings Usman Ansari, 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: Author Usman Ansari <ua1422 at gmail.com> needs to sign off.
WARNING: Unexpected sign-offs from developers who are not authors or co-authors or committers: Usman Ansari <u1422 at gmail.com>
WARNING: Line is 85 characters long (recommended limit is 79)
#28 FILE: include/openvswitch/hmap.h:139:
         (NODE != OBJECT_CONTAINING(NULL, NODE, MEMBER)) || ((NODE = NULL), false); \

WARNING: Line is 85 characters long (recommended limit is 79)
#34 FILE: include/openvswitch/hmap.h:144:
         (NODE != OBJECT_CONTAINING(NULL, NODE, MEMBER)) || ((NODE = NULL), false); \

WARNING: Line is 85 characters long (recommended limit is 79)
#43 FILE: include/openvswitch/hmap.h:173:
         (NODE != OBJECT_CONTAINING(NULL, NODE, MEMBER)) || ((NODE = NULL), false); \

WARNING: Line is 85 characters long (recommended limit is 79)
#52 FILE: include/openvswitch/hmap.h:182:
         ((NODE != OBJECT_CONTAINING(NULL, NODE, MEMBER)) || ((NODE = NULL), false) \

WARNING: Line is 85 characters long (recommended limit is 79)
#61 FILE: include/openvswitch/hmap.h:193:
         (NODE != OBJECT_CONTAINING(NULL, NODE, MEMBER)) || ((NODE = NULL), false); \

WARNING: Line is 84 characters long (recommended limit is 79)
#70 FILE: include/openvswitch/hmap.h:214:
         (NODE != OBJECT_CONTAINING(NULL, NODE, MEMBER)) || ((NODE = NULL), false);)

Lines checked: 76, Warnings: 7, Errors: 1


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