[ovs-dev] [PATCH ovn 02/21] ovn-nbctl: Refactor into infrastructure and northbound details.

0-day Robot robot at bytheb.org
Sat Mar 27 01:00:47 UTC 2021


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:
ERROR: Use ovs_abort() in place of abort()
#605 FILE: utilities/ovn-dbctl.c:565:
            abort();

WARNING: Line has non-spaces leading whitespace
WARNING: Line has trailing whitespace
#820 FILE: utilities/ovn-dbctl.c:780:
    

WARNING: Line lacks whitespace around operator
#933 FILE: utilities/ovn-dbctl.c:893:
                                  argv[optind-1]);

WARNING: Line lacks whitespace around operator
#944 FILE: utilities/ovn-dbctl.c:904:
                              argv[optind-1]);

WARNING: Comment with 'xxx' marker
#1380 FILE: utilities/ovn-nbctl.c:78:
        /* XXX add verification that table is empty */

Lines checked: 2779, Warnings: 5, 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