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

Commit 960f0b19 authored by Gregory Bean's avatar Gregory Bean Committed by Bryan Huntsman
Browse files

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: default avatarGregory Bean <gbean@codeaurora.org>
(cherry picked from commit 8e6b9d3790595198a34320f1c3f4504cd258fed1)
Signed-off-by: default avatarStepan Moskovchenko <stepanm@codeaurora.org>
Signed-off-by: default avatarRishabh Bhatnagar <rishabhb@codeaurora.org>
parent 6ceb5dda

scripts/checkpatch.pl

+14 −7
Original line number Diff line number Diff line
@@ -2508,14 +2508,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;