[coreboot-gerrit] Change in coreboot[master]: util/lint: Update checkpatch max line length from 80 to 96

Martin Roth (Code Review) gerrit at coreboot.org
Wed Jul 18 17:26:42 CEST 2018


Martin Roth has uploaded this change for review. ( https://review.coreboot.org/27533


Change subject: util/lint: Update checkpatch max line length from 80 to 96
......................................................................

util/lint: Update checkpatch max line length from 80 to 96

There are arguments for and against the 80 character line length.  I
don't think there are significant arguments for making it shorter, but
many people would prefer that it was much longer.

This patch is trying to be a compromise between the two groups without
adjusting the tab length.

Change-Id: Ic76f9544a609f4c941902c45d5f77df0b644f6a8
Signed-off-by: Martin Roth <martin at coreboot.org>
---
M util/lint/checkpatch.pl
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/33/27533/1

diff --git a/util/lint/checkpatch.pl b/util/lint/checkpatch.pl
index c35a0d6..ceeda48 100755
--- a/util/lint/checkpatch.pl
+++ b/util/lint/checkpatch.pl
@@ -49,7 +49,7 @@
 my @exclude = (); #coreboot
 my $help = 0;
 my $configuration_file = ".checkpatch.conf";
-my $max_line_length = 80;
+my $max_line_length = 96;	# coreboot - 80 characters + 2 tabs indentation
 my $ignore_perl_version = 0;
 my $minimum_perl_version = 5.10.0;
 my $min_conf_desc_length = 4;

-- 
To view, visit https://review.coreboot.org/27533
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic76f9544a609f4c941902c45d5f77df0b644f6a8
Gerrit-Change-Number: 27533
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Roth <martinroth at google.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180718/9fce567c/attachment.html>


More information about the coreboot-gerrit mailing list