[ovs-dev] [PATCH] checkpatch: Increase recommended line length to 100

Ben Pfaff blp at ovn.org
Thu Jun 18 16:51:35 UTC 2020


On Thu, Jun 11, 2020 at 01:09:32PM +0300, Roi Dayan wrote:
> This is to match a recent kernel checkpatch change that
> also increased it to 100 line length.
> 
> Signed-off-by: Roi Dayan <roid at mellanox.com>

I don't know what you effect you want from this change.

If you want to keep checkpatch from complaining for long lines in kernel
code, then this is not the right way, because checkpatch ignores the
datapath/ directory that contains kernel code.

If you want to update OVS coding style to allow longer lines in OVS's
own code, then the patch should update
Documentation/internals/contributing/coding-style.rst
in a few different places, and we should probably discuss it more
extensively than just a bare ack.


More information about the dev mailing list