Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34386 )
Change subject: util/lint/checkpatch.pl: Update line length to 96 ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/34386/1/util/lint/checkpatch.pl File util/lint/checkpatch.pl:
https://review.coreboot.org/c/coreboot/+/34386/1/util/lint/checkpatch.pl@54 PS1, Line 54: my $max_line_length = 96;
I think we can just add --max-line-length 96 to the coreboot/checkpatch.conf file.
In fact it's already there. See CB:33405. And I can confirm that there've been no more warnings for 80 char violations recently, I'm pretty sure it works.
(BTW, do we have an ETA for when the code base will be reformatted? The current state is super annoying -- I keep having to find line length violations manually and ask people to please stick to the rest of the file's style until it is converted in full. We just went from having *some* enforced standard to no enforced standard at all, which I'm pretty sure is not what people wanted. You guys were very passionate about making this decision, now please actually implement it too.)