Numan Siddique <nusid...@redhat.com> writes: > On Wed, Feb 27, 2019 at 4:48 PM 0-day Robot <ro...@bytheb.org> wrote: > > Bleep bloop. Greetings Numan Siddique, 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 lacks whitespace around operator > #935 FILE: ovn/utilities/ovn-nbctl.c:697: > ha-chassis-group-add GRP Create an HA chassis group GRP\n\ > > WARNING: Line lacks whitespace around operator > #936 FILE: ovn/utilities/ovn-nbctl.c:698: > ha-chassis-group-del GRP Delete the HA chassis group GRP\n\ > > WARNING: Line lacks whitespace around operator > #937 FILE: ovn/utilities/ovn-nbctl.c:699: > ha-chassis-group-list List the HA chassis groups\n\ > > WARNING: Line lacks whitespace around operator > #938 FILE: ovn/utilities/ovn-nbctl.c:700: > ha-chassis-group-add-chassis GRP CHASSIS [PRIORITY] Adds an HA\ > > WARNING: Line lacks whitespace around operator > #940 FILE: ovn/utilities/ovn-nbctl.c:702: > ha-chassis-group-del-chassis GRP CHASSIS Deletes the HA chassis\ > > Lines checked: 1544, Warnings: 5, Errors: 0 > > Hi Aaron, > > I couldn't resolve this warning. > I see this warning even with the existing code here - > https://github.com/openvswitch/ovs/blob/master/ovn/utilities/ovn-nbctl.c#L706 > > Any suggestions how to fix. Or we can ignore this.
We can ignore it. Still need to fix up checkpatch to ignore these whitespace checks within strings. > Thanks > Numan > > Please check this out. If you feel there has been an error, please email > acon...@bytheb.org > > Thanks, > 0-day Robot _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
