diff options
author | Julius Werner <jwerner@chromium.org> | 2016-08-03 20:07:35 -0700 |
---|---|---|
committer | Julius Werner <jwerner@chromium.org> | 2016-08-05 00:01:55 +0200 |
commit | 6e05c33626e128c383470579f423b1ee569302ba (patch) | |
tree | b43b69620cd820751487532466f3d9f5cfae388a | |
parent | 88a25ec20a27c8f2131a19caae95e6109fecc10a (diff) |
checkpatch: Add BLOCK_COMMENT_STYLE to ignore list
Linux' newest checkpatch.pl flags comments like these:
/* This is a concise 2-line comment that explains what the code does in
* sufficient detail without wasting too much vertical space. */
do_stuff_that_needs_explaining();
Comments like these have been used in our code base for a long time and
I don't think we should disallow them now. Ending the comment on the
same line doesn't really hurt readability and wastes less screen real
estate (which in turn usually helps overall code comprehension).
Change-Id: Ifd57f3d3a62738165024cb4b2e75a4f815a57922
Signed-off-by: Julius Werner <jwerner@chromium.org>
Reviewed-on: https://review.coreboot.org/16060
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
-rw-r--r-- | .checkpatch.conf | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/.checkpatch.conf b/.checkpatch.conf index 4b88745ba2..f7b9b32519 100644 --- a/.checkpatch.conf +++ b/.checkpatch.conf @@ -14,3 +14,4 @@ --ignore PREFER_PACKED --ignore PREFER_PRINTF --ignore SPLIT_STRING +--ignore BLOCK_COMMENT_STYLE |