Bleep bloop. Greetings Aleksandr Smirnov, 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
#507 FILE: northd/en-ls-arp.h:72:
HMAP_FOR_EACH (LS_ARP_REC, key_node, &(TABLE)->entries)
ERROR: Inappropriate spacing in pointer declaration
WARNING: Line lacks whitespace around operator
#522 FILE: northd/en-ls-arp.h:87:
void *en_ls_arp_init(struct engine_node*, struct engine_arg*);
ERROR: Inappropriate spacing in pointer declaration
WARNING: Line lacks whitespace around operator
#523 FILE: northd/en-ls-arp.h:88:
void en_ls_arp_cleanup(void* data);
ERROR: Inappropriate spacing in pointer declaration
WARNING: Line lacks whitespace around operator
#524 FILE: northd/en-ls-arp.h:89:
void en_ls_arp_clear_tracked_data(void* data);
ERROR: Inappropriate spacing in pointer declaration
WARNING: Line lacks whitespace around operator
#525 FILE: northd/en-ls-arp.h:90:
enum engine_node_state en_ls_arp_run(struct engine_node*, void* data);
ERROR: Improper whitespace around control block
#734 FILE: northd/northd.c:9298:
SSET_FOR_EACH(item, &port_check) {
ERROR: Improper whitespace around control block
#740 FILE: northd/northd.c:9304:
SSET_FOR_EACH(item, &resident_check) {
WARNING: Line is 84 characters long (recommended limit is 79)
#807 FILE: northd/northd.c:17980:
build_lswitch_arp_chassis_resident(od, lsi->lflows,
ls_arp_rec);
Lines checked: 1164, Warnings: 5, Errors: 7
Please check this out. If you feel there has been an error, please email
acon...@redhat.com
Thanks,
0-day Robot
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
