diff options
| author | Steve Muckle <smuckle@codeaurora.org> | 2012-03-09 11:15:24 -0800 |
|---|---|---|
| committer | David Keitel <dkeitel@codeaurora.org> | 2016-03-23 20:57:38 -0700 |
| commit | d680dbcb3ab0573d4fb95114a13a2dae1a52a8ef (patch) | |
| tree | 307c6fadcd483d0438a525249e4adc4c3ce3a3a0 /scripts/checkpatch.pl | |
| parent | 30b0925570a76a39aa750865ecb105beb5cb0d68 (diff) | |
checkpatch: recognize only specific tags when ending commit text
There are some very frequently used tags that checkpatch can look
for as an ending to the commit text rather than using a pattern,
which can generate false positives in the "no commit text" rule.
Change-Id: I5b4400017b8273bcd9f5a59b3e28965c0062bef4
Signed-off-by: Steve Muckle <smuckle@codeaurora.org>
Signed-off-by: Stepan Moskovchenko <stepanm@codeaurora.org>
Diffstat (limited to 'scripts/checkpatch.pl')
| -rwxr-xr-x | scripts/checkpatch.pl | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 0b1da75d24c1..dbe77d70aa60 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -2265,7 +2265,10 @@ sub process { "commit text line over " . SHORTTEXT_LIMIT . " characters\n" . $herecurr); - } elsif ($line=~/^\s*[\x21-\x39\x3b-\x7e]+:/) { + } elsif ($line=~/^\s*change-id:/i || + $line=~/^\s*signed-off-by:/i || + $line=~/^\s*crs-fixed:/i || + $line=~/^\s*acked-by:/i) { # this is a tag, there must be commit # text by now if ($commit_text_present == 0) { |