Bleep bloop.  Greetings Adrian Moreno, 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: Inappropriate bracing around statement
#88 FILE: include/openvswitch/hmap.h:182:
    HMAP_FOR_EACH_SAFE_LONG_INIT (NODE, NEXT, MEMBER, HMAP, (void) NEXT)

ERROR: Inappropriate bracing around statement
#103 FILE: include/openvswitch/hmap.h:195:
    HMAP_FOR_EACH_SAFE_SHORT_INIT (NODE, MEMBER, HMAP, (void) 0)

ERROR: Use ovs_assert() in place of assert()
#1546 FILE: tests/test-hmap.c:277:
            assert(n == n_remaining);

ERROR: Use ovs_assert() in place of assert()
#1555 FILE: tests/test-hmap.c:286:
                assert(i < n);

ERROR: Use ovs_assert() in place of assert()
#1560 FILE: tests/test-hmap.c:291:
                        assert(j < n_remaining);

ERROR: Use ovs_assert() in place of assert()
#1570 FILE: tests/test-hmap.c:301:
            assert(i == n);

ERROR: Use ovs_assert() in place of assert()
#1571 FILE: tests/test-hmap.c:302:
            assert(e == NULL);

Lines checked: 1842, Warnings: 0, Errors: 7


Please check this out.  If you feel there has been an error, please email 
[email protected]

Thanks,
0-day Robot
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to