Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 869c4b11 authored by Gregory Bean's avatar Gregory Bean Committed by Kyle Yan
Browse files

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.

Signed-off-by: default avatarGregory Bean <gbean@codeaurora.org>
(cherry picked from commit 8e6b9d3790595198a34320f1c3f4504cd258fed1)
Signed-off-by: default avatarStepan Moskovchenko <stepanm@codeaurora.org>
parent 568293b8

scripts/checkpatch.pl

+14 −7
Original line number Diff line number Diff line
@@ -2270,14 +2270,21 @@ sub process { 
					     " characters\n" . $herecurr);

				}

			} elsif ($shorttext == CHECK_NEXT_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");

						     "non-blank line after " .

						     "summary line\n" .

						     $sublinenr . $here .

						     "\n" . $subjectline .

						     "\n" . $line . "\n");

					}

				}

			} elsif ($line=~/^Subject: \[[^\]]*\] (.*)/) {

				$shorttext = CHECK_NEXT_SHORTTEXT;