Commit 42d7523c authored May 01, 2017 by Vikram Mulukutla Committed by Prasad Sodagudi Jul 06, 2017

checkpatch: Use upstream commit text length checks



Upstream has merged a few commit text length checks
that make our own checks redundant. Furthermore,
upstream also has a newer exception list (such as
allowing Fixes: lines to be >75 characters) that
we should take advantage of.

Change-Id: I894482c1a4182ad71146d5d433610d4cb210cb1d
Signed-off-by: Vikram Mulukutla <markivx@codeaurora.org>
Signed-off-by: Prasad Sodagudi <psodagud@codeaurora.org>

parent b2e19b32

scripts/checkpatch.pl

+0 −9

Original line number	Diff line number	Diff line
		@@ -2463,15 +2463,6 @@ sub process {
		$herecurr);
		}
		$shorttext = AFTER_SHORTTEXT;
		} elsif (length($line) > (SHORTTEXT_LIMIT +
		$shorttext_exspc)
		&& $line !~ /^:([0-7]{6}\s){2}
		([[:xdigit:]]+\.*
		\s){2}\w+\s\w+/xms) {
		WARN("LONG_COMMIT_TEXT",
		"commit text line over " .
		SHORTTEXT_LIMIT .
		" characters\n" . $herecurr);
		} elsif ($line=~/^\s*change-id:/i \|\|
		$line=~/^\s*signed-off-by:/i \|\|
		$line=~/^\s*crs-fixed:/i \|\|