Re: [Qemu-devel] [PATCH 2/3] checkpatch: bump most warnings to errors

[Cornelia Huck]

On Wed, 10 Aug 2016 03:08:33 -0400 (EDT)
Paolo Bonzini <address@hidden> wrote:

> > Paolo Bonzini <address@hidden> writes:
> > 
> > > This only leaves a warning-level message for extra-long lines, which
> > > are relatively common and cause patchew to send email that will likely
> > > be ignored.
> > >
> > > Signed-off-by: Paolo Bonzini <address@hidden>
> > 
> > Are we ready to give up on illegibly long lines?
> 
> We have other levels of code review than checkpatch.  80 chars can be
> illegibly short in some circumstances where 83 or 84 are enough.

We could leave the WARN at 80 chars and add an error at 120 chars or so.