diff options
author | Julius Werner <jwerner@chromium.org> | 2021-03-25 21:12:49 -0700 |
---|---|---|
committer | Julius Werner <jwerner@chromium.org> | 2021-06-08 22:10:07 +0000 |
commit | 218906df66987ff0664dff986996be3df4bbde24 (patch) | |
tree | 2e619a24a37f5ec153d4b0c291b84abaa58587dc /util/lint/remccoms3.sed | |
parent | aee70a8a8effda2ce795b3dde3f2b780c90e562b (diff) |
lint: checkpatch: Add SUSPICIOUS_CODE_INDENT test
This patch adds a new test to checkpatch that identifies cases where a
line after a conditional statement is incorrectly intended (possibly
indicating the mistake of forgetting to add braces), like this:
if (a)
b;
c;
Unfortunately, it seems like checkpatch is partially unmaintained in
upstream Linux at the moment with maintainers either not responding at
all or not even willing to look at new patches [1]. Since detecting this
error class is important to coreboot, let's just carry this feature
locally for now.
[1] https://lkml.org/lkml/2021/4/15/1488
Signed-off-by: Julius Werner <jwerner@chromium.org>
Change-Id: I7bb90b56dfc7582271d2b82cb42a2c1df477054f
Reviewed-on: https://review.coreboot.org/c/coreboot/+/51838
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Menzel <paulepanter@mailbox.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'util/lint/remccoms3.sed')
0 files changed, 0 insertions, 0 deletions