diff options
| author | Matt Wagantall <mattw@codeaurora.org> | 2012-08-18 11:21:00 -0700 |
|---|---|---|
| committer | David Keitel <dkeitel@codeaurora.org> | 2016-03-23 20:57:39 -0700 |
| commit | 5cc22e2128447488ad7fc22d25a57fb9d54614d3 (patch) | |
| tree | 7d233445dfdbad0178b71272f9e32edd26237a89 | |
| parent | d680dbcb3ab0573d4fb95114a13a2dae1a52a8ef (diff) | |
checkpatch: Excuse reverts from "summary line over 75 characters" check
Rather than forcing authors of reverts to truncate the summary line
because the "Revert" prefix added pushes it over the character limit,
excuse reverts from this rule.
Change-Id: I395dfff3327e360ef935d4a685c38df6577e3867
Signed-off-by: Matt Wagantall <mattw@codeaurora.org>
Signed-off-by: Stepan Moskovchenko <stepanm@codeaurora.org>
| -rwxr-xr-x | scripts/checkpatch.pl | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index dbe77d70aa60..17a3a6d1785c 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -2316,7 +2316,7 @@ sub process { $subjectline = $line; $sublinenr = "#$linenr & "; # Check for Subject line less than line limit - if (length($1) > SHORTTEXT_LIMIT) { + if (length($1) > SHORTTEXT_LIMIT && !($1 =~ m/Revert\ \"/)) { WARN("LONG_SUMMARY_LINE", "summary line over " . SHORTTEXT_LIMIT . @@ -2328,7 +2328,7 @@ sub process { # headers so we are now in the shorttext. $shorttext = IN_SHORTTEXT_BLANKLINE; $shorttext_exspc = 4; - if (length($1) > SHORTTEXT_LIMIT) { + if (length($1) > SHORTTEXT_LIMIT && !($1 =~ m/Revert\ \"/)) { WARN("LONG_SUMMARY_LINE", "summary line over " . SHORTTEXT_LIMIT . |