checkpatch: Don't complain about MIME headers.
When patches contain extended character sets, patches will contain MIME headers after the subject line, which should not be confused for a too-long summary line. Change-Id: I66f70b465ef804444a3bdb49867654b5613b13a7 Signed-off-by: Gregory Bean <gbean@codeaurora.org> (cherry picked from commit 8e6b9d3790595198a34320f1c3f4504cd258fed1) Signed-off-by: Stepan Moskovchenko <stepanm@codeaurora.org> Signed-off-by: Rishabh Bhatnagar <rishabhb@codeaurora.org>
This commit is contained in:
parent
a2d6e2ef6e
commit
a8c5a946e9
@ -2582,14 +2582,21 @@ sub process {
|
||||
" characters\n" . $herecurr);
|
||||
}
|
||||
} elsif ($shorttext == CHECK_NEXT_SHORTTEXT) {
|
||||
$shorttext = IN_SHORTTEXT;
|
||||
# The Subject line doesn't have to be the last header in the patch.
|
||||
# Avoid moving to the IN_SHORTTEXT state until clear of all headers.
|
||||
# Per RFC5322, continuation lines must be folded, so any left-justified
|
||||
# text which looks like a header is definitely a header.
|
||||
if ($line!~/^[\x21-\x39\x3b-\x7e]+:/) {
|
||||
$shorttext = IN_SHORTTEXT;
|
||||
# Check for Subject line followed by a blank line.
|
||||
if (length($line) != 0) {
|
||||
WARN("NONBLANK_AFTER_SUMMARY",
|
||||
"non-blank line after summary " .
|
||||
"line\n" . $sublinenr . $here .
|
||||
"\n" . $subjectline . "\n" .
|
||||
$line . "\n");
|
||||
if (length($line) != 0) {
|
||||
WARN("NONBLANK_AFTER_SUMMARY",
|
||||
"non-blank line after " .
|
||||
"summary line\n" .
|
||||
$sublinenr . $here .
|
||||
"\n" . $subjectline .
|
||||
"\n" . $line . "\n");
|
||||
}
|
||||
}
|
||||
} elsif ($line=~/^Subject: \[[^\]]*\] (.*)/) {
|
||||
$shorttext = CHECK_NEXT_SHORTTEXT;
|
||||
|
Loading…
Reference in New Issue
Block a user