| Message ID | 1425574649.31859.14.camel@perches.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Thu, 05 Mar 2015, Joe Perches <joe@perches.com> wrote: > Only commit log and patch additions are checked for > typos and spelling errors currently. Add a check > of the email subject line too. > > Suggested-by: Jani Nikula <jani.nikula@linux.intel.com> > Signed-off-by: Joe Perches <joe@perches.com> Thanks Joe. FWIW, Tested-by: Jani Nikula <jani.nikula@intel.com> > --- > scripts/checkpatch.pl | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl > index 421bbb4..c061a63 100755 > --- a/scripts/checkpatch.pl > +++ b/scripts/checkpatch.pl > @@ -2303,7 +2303,8 @@ sub process { > } > > # Check for various typo / spelling mistakes > - if (defined($misspellings) && ($in_commit_log || $line =~ /^\+/)) { > + if (defined($misspellings) && > + ($in_commit_log || $line =~ /^(?:\+|Subject:)/i)) { > while ($rawline =~ /(?:^|[^a-z@])($misspellings)(?:$|[^a-z@])/gi) { > my $typo = $1; > my $typo_fix = $spelling_fix{lc($typo)}; > >
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 421bbb4..c061a63 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -2303,7 +2303,8 @@ sub process { } # Check for various typo / spelling mistakes - if (defined($misspellings) && ($in_commit_log || $line =~ /^\+/)) { + if (defined($misspellings) && + ($in_commit_log || $line =~ /^(?:\+|Subject:)/i)) { while ($rawline =~ /(?:^|[^a-z@])($misspellings)(?:$|[^a-z@])/gi) { my $typo = $1; my $typo_fix = $spelling_fix{lc($typo)};
Only commit log and patch additions are checked for typos and spelling errors currently. Add a check of the email subject line too. Suggested-by: Jani Nikula <jani.nikula@linux.intel.com> Signed-off-by: Joe Perches <joe@perches.com> --- scripts/checkpatch.pl | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)