diff options
| -rwxr-xr-x | util/lint/checkpatch.pl | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/util/lint/checkpatch.pl b/util/lint/checkpatch.pl index 0e2988ea9b..ce56feb6cc 100755 --- a/util/lint/checkpatch.pl +++ b/util/lint/checkpatch.pl @@ -2854,6 +2854,7 @@ sub process { # coreboot: The line subject limit is 65 # Check for line lengths > 65 in commit subject +# coreboot: Check for a period at the end of the subject line if ($in_header_lines && $line =~ /^Subject: /) { $line = $line.$rawlines[$linenr]; @@ -2862,6 +2863,10 @@ sub process { WARN("COMMIT_LOG_LONG_LINE", "Possible long commit subject (prefer a maximum 65 characters)\n" . $herecurr); } + if ($line =~ /\.$/) { + WARN("SUBJECT_HAS_FULL_STOP", + "Subject line should not end with a period.\n" . $herecurr); + } } # Reset possible stack dump if a blank line is found |