UPSTREAM: 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).

BUG=None
BRANCH=None
TEST=None

Change-Id: I4d0d7c166c3ecd81a8a0d8a4c5133414c9cb5841
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>
Reviewed-on: https://chromium-review.googlesource.com/368401
Commit-Ready: Furquan Shaikh <furquan@chromium.org>
Tested-by: Furquan Shaikh <furquan@chromium.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
This commit is contained in:
Julius Werner 2016-08-03 20:07:35 -07:00 committed by chrome-bot
parent 5d5421b7a2
commit e2a5efc68a

View file

@ -14,3 +14,4 @@
--ignore PREFER_PACKED --ignore PREFER_PACKED
--ignore PREFER_PRINTF --ignore PREFER_PRINTF
--ignore SPLIT_STRING --ignore SPLIT_STRING
--ignore BLOCK_COMMENT_STYLE