Bleep bloop. Greetings Ben Pfaff, 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:
WARNING: Line is 80 characters long (recommended limit is 79)
#148 FILE: lib/ovsdb-cs.c:159:
int max_version; /* Maximum version of monitor request to use. */
WARNING: Comment with 'xxx' marker
#301 FILE: lib/ovsdb-cs.c:312:
* XXX 'max_version' should ordinarily be 3, to allow use of the most efficient
WARNING: Comment with 'xxx' marker
#576 FILE: lib/ovsdb-cs.c:587:
* levels. XXX?) */
WARNING: Comment with 'xxx' marker
#1413 FILE: lib/ovsdb-cs.c:1424:
/* XXX handle failure */
WARNING: Comment with 'xxx' marker
#1952 FILE: lib/ovsdb-cs.c:1963:
/* XXX return failure? */
ERROR: Improper whitespace around control block
#4236 FILE: lib/ovsdb-idl.c:1256:
LIST_FOR_EACH_SAFE(row, next, track_node, &table->track_list) {
WARNING: Comment with 'xxx' marker
#4378 FILE: lib/ovsdb-idl.c:1331:
/* XXX This isn't semantically required, because we only need to
Lines checked: 5058, Warnings: 6, Errors: 1
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
