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

[0-day Robot]

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()
#606 FILE: utilities/ovn-dbctl.c:566:
            abort();

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

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

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

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

Lines checked: 2781, 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