[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/3] checkpatch: bump most warnings to errors
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 2/3] checkpatch: bump most warnings to errors |
Date: |
Wed, 10 Aug 2016 03:08:33 -0400 (EDT) |
> 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.
Paolo
> What happened to "[PATCH 2/4] CODING_STYLE, checkpatch: update line
> length rules"?
[Qemu-devel] [PATCH 3/3] checkpatch: default to success if only warnings, Paolo Bonzini, 2016/08/09
Re: [Qemu-devel] [PATCH 0/3] checkpatch tweaks, Fam Zheng, 2016/08/09
Re: [Qemu-devel] [PATCH 0/3] checkpatch tweaks, Cornelia Huck, 2016/08/10