[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/4] CODING_STYLE, checkpatch: update line lengt
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 2/4] CODING_STYLE, checkpatch: update line length rules |
Date: |
Mon, 07 Sep 2015 19:05:38 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Paolo Bonzini <address@hidden> writes:
> On 07/09/2015 17:23, Markus Armbruster wrote:
>> > Apart from copy-n-pasting, there is also the problem that you can run
>> > "checkpatch.pl -f" on a whole file ... it would also be ugly to suddenly
>> > have (much) more warnings here.
>>
>> Feature. If you run checkpatch on a whole file, you obviously do it to
>> find its ugly spots. Lines longer than 76 characters qualify.
>
> Based on the statistics, half of QEMU's files has at least one 76-79
> character line. The noise from checkpatch.pl -f is actually a worse
> thing than the cut-and-paste, but that's something that can be fixed in
> other ways (e.g. different strictness for checkpatch.pl vs.
> checkpatch.pl -f).
Yup.
> That said, and even though Thomas obviously hasn't read the previous
> discussion, :) I do believe that 76 characters is too strict a limit.
It's not a strict limit, it's a warning. The strict limit is 90.
> 76 would be great (two levels of email quoting are what you get 99% of
> the time), and 78 would be nice, but I believe 79 provides the biggest
> bang for the buck.
78 gives one level of quoting, and two-way diffs.
- [Qemu-devel] [PATCH 0/4] updates to coding style and checkpatch, Paolo Bonzini, 2015/09/07
- [Qemu-devel] [PATCH 1/4] CODING_STYLE: update mixed declaration rules, Paolo Bonzini, 2015/09/07
- [Qemu-devel] [PATCH 2/4] CODING_STYLE, checkpatch: update line length rules, Paolo Bonzini, 2015/09/07
- Re: [Qemu-devel] [PATCH 2/4] CODING_STYLE, checkpatch: update line length rules, Thomas Huth, 2015/09/07
- Re: [Qemu-devel] [PATCH 2/4] CODING_STYLE, checkpatch: update line length rules, Paolo Bonzini, 2015/09/07
- Re: [Qemu-devel] [PATCH 2/4] CODING_STYLE, checkpatch: update line length rules, Thomas Huth, 2015/09/07
- Re: [Qemu-devel] [PATCH 2/4] CODING_STYLE, checkpatch: update line length rules, Markus Armbruster, 2015/09/07
- Re: [Qemu-devel] [PATCH 2/4] CODING_STYLE, checkpatch: update line length rules, Paolo Bonzini, 2015/09/07
- Re: [Qemu-devel] [PATCH 2/4] CODING_STYLE, checkpatch: update line length rules,
Markus Armbruster <=
- Re: [Qemu-devel] [PATCH 2/4] CODING_STYLE, checkpatch: update line length rules, John Snow, 2015/09/09
- Re: [Qemu-devel] [PATCH 2/4] CODING_STYLE, checkpatch: update line length rules, Peter Maydell, 2015/09/09
- Re: [Qemu-devel] [PATCH 2/4] CODING_STYLE, checkpatch: update line length rules, Eduardo Habkost, 2015/09/09
- Re: [Qemu-devel] [PATCH 2/4] CODING_STYLE, checkpatch: update line length rules, Paolo Bonzini, 2015/09/09
- Re: [Qemu-devel] [PATCH 2/4] CODING_STYLE, checkpatch: update line length rules, Eduardo Habkost, 2015/09/09
- Re: [Qemu-devel] [PATCH 2/4] CODING_STYLE, checkpatch: update line length rules, Thomas Huth, 2015/09/07
- Re: [Qemu-devel] [PATCH 2/4] CODING_STYLE, checkpatch: update line length rules, Eduardo Habkost, 2015/09/09
Re: [Qemu-devel] [PATCH 2/4] CODING_STYLE, checkpatch: update line length rules, Markus Armbruster, 2015/09/07