On 2020-06-02 1:35 AM, 0-day Robot wrote:
> Bleep bloop. Greetings Roi Dayan, 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)
> #252 FILE: lib/netdev-offload-tc.c:1824:
> parse_tc_flower_to_match(&flower, match, actions, stats, attrs, buf,
> false);
>
> Lines checked: 368, Warnings: 1, Errors: 0
>
>
> Please check this out. If you feel there has been an error, please email
> acon...@redhat.com
>
> Thanks,
> 0-day Robot
>
it is pretty short line so we didn't want to do redundant line breaks.
is it important or can we skip this?
should we maybe update ovs checkpatch to 100 chars to match
linux kernel? i guess the current 79 is also started because
the kernel was 79?
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
