Commit 29ee1b0c authored by Joe Perches's avatar Joe Perches Committed by Linus Torvalds

checkpatch: ignore email headers better

There are some patches created by git format-patch that when scanned by
checkpatch report errors on lines like

To:	address.tld

This is a checkpatch false positive.

Improve the logic a bit to ignore folded email headers to avoid emitting
these messages.
Signed-off-by: default avatarJoe Perches <joe@perches.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 5a4e1fd3
...@@ -1637,7 +1637,7 @@ sub process { ...@@ -1637,7 +1637,7 @@ sub process {
my $signoff = 0; my $signoff = 0;
my $is_patch = 0; my $is_patch = 0;
my $in_header_lines = 1; my $in_header_lines = $file ? 0 : 1;
my $in_commit_log = 0; #Scanning lines before patch my $in_commit_log = 0; #Scanning lines before patch
my $non_utf8_charset = 0; my $non_utf8_charset = 0;
...@@ -1993,7 +1993,8 @@ sub process { ...@@ -1993,7 +1993,8 @@ sub process {
# Check if it's the start of a commit log # Check if it's the start of a commit log
# (not a header line and we haven't seen the patch filename) # (not a header line and we haven't seen the patch filename)
if ($in_header_lines && $realfile =~ /^$/ && if ($in_header_lines && $realfile =~ /^$/ &&
$rawline !~ /^(commit\b|from\b|[\w-]+:).+$/i) { !($rawline =~ /^\s+\S/ ||
$rawline =~ /^(commit\b|from\b|[\w-]+:).*$/i)) {
$in_header_lines = 0; $in_header_lines = 0;
$in_commit_log = 1; $in_commit_log = 1;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment