Patrick Georgi has uploaded this change for review.
lint/clang-format: set to 96 chars per line
80 chars + 2 tabs was the compromise we got to in the last round of
discussion.
Change-Id: I9293a69d1bea900da36501cde512004d0695ad37
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
---
M .clang-format
M util/lint/lint-007-checkpatch
2 files changed, 5 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/51/31651/1
diff --git a/.clang-format b/.clang-format
index d853f50..5c8aa3c 100644
--- a/.clang-format
+++ b/.clang-format
@@ -7,7 +7,7 @@
IndentCaseLabels: false
SortIncludes: false
ContinuationIndentWidth: 8
-ColumnLimit: 0
+ColumnLimit: 96
AlwaysBreakBeforeMultilineStrings: true
AllowShortLoopsOnASingleLine: false
AllowShortFunctionsOnASingleLine: false
diff --git a/util/lint/lint-007-checkpatch b/util/lint/lint-007-checkpatch
index afa593e..a7b63e8 100755
--- a/util/lint/lint-007-checkpatch
+++ b/util/lint/lint-007-checkpatch
@@ -28,6 +28,8 @@
^src/vendorcode\|\
^Documentation"
+opts="--max-line-length 96"
+
# default: test src and util
if [ "$1" = "" ]; then
INCLUDED_DIRS="src util"
@@ -35,7 +37,7 @@
elif [ "$1" = "diff" ]; then
args=$( echo $EXCLUDED_DIRS | \
sed -e 's,\\|, ,g' -e 's,\^,--exclude=,g' )
- util/lint/checkpatch.pl --quiet --no-signoff $args -
+ util/lint/checkpatch.pl --quiet --no-signoff $opts $args -
exit $?
# Space separated list of directories to test
else
@@ -49,5 +51,5 @@
grep -v $EXCLUDED_DIRS )
for FILE in $FILELIST; do
- util/lint/checkpatch.pl --show-types --file --quiet "$FILE"
+ util/lint/checkpatch.pl --show-types --file --quiet $opts "$FILE"
done
To view, visit change 31651. To unsubscribe, or for help writing mail filters, visit settings.