On Tue, Dec 13, 2022 at 3:33 PM Ihar Hrachyshka <[email protected]> wrote: > > Abhiram, these style violations should be fixed.
Hi Ihar, These warnings are ok and can be ignored. Perhaps we need to fix it in the checkpatch to address these warnings. Numan > > (I will review the actual code changes tomorrow.) > > On Tue, Dec 13, 2022 at 2:40 PM 0-day Robot <[email protected]> wrote: > > > > Bleep bloop. Greetings Abhiram R N, 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 > > #336 FILE: utilities/ovn-nbctl.c:276: > > mirror-add NAME TYPE INDEX FILTER IP\n\ > > > > WARNING: Line lacks whitespace around operator > > #345 FILE: utilities/ovn-nbctl.c:285: > > mirror-del [NAME] remove mirrors\n\ > > > > WARNING: Line lacks whitespace around operator > > #346 FILE: utilities/ovn-nbctl.c:286: > > mirror-list print mirrors\n\ > > > > WARNING: Line lacks whitespace around operator > > #355 FILE: utilities/ovn-nbctl.c:328: > > lsp-attach-mirror PORT MIRROR attach source PORT to MIRROR\n\ > > > > WARNING: Line lacks whitespace around operator > > #356 FILE: utilities/ovn-nbctl.c:329: > > lsp-detach-mirror PORT MIRROR detach source PORT from MIRROR\n\ > > > > Lines checked: 710, Warnings: 5, Errors: 0 > > > > > > 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 > > > > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
