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

Commit 187016ef authored by Matt Wagantall's avatar Matt Wagantall Committed by Kyle Yan
Browse files

checkpatch: Excuse reverts from "summary line over 75 characters" check 



Rather than forcing authors of reverts to truncate the summary line
because the "Revert" prefix added pushes it over the character limit,
excuse reverts from this rule.

Change-Id: I395dfff3327e360ef935d4a685c38df6577e3867
Signed-off-by: default avatarMatt Wagantall <mattw@codeaurora.org>
Signed-off-by: default avatarStepan Moskovchenko <stepanm@codeaurora.org>
parent 317c2c79

scripts/checkpatch.pl

+2 −2
Original line number Original line Diff line number Diff line
@@ -2339,7 +2339,7 @@ sub process { 
				$subjectline = $line;

				$subjectline = $line;

				$sublinenr = "#$linenr & ";

				$sublinenr = "#$linenr & ";

# Check for Subject line less than line limit

# Check for Subject line less than line limit

				if (length($1) > SHORTTEXT_LIMIT) {

				if (length($1) > SHORTTEXT_LIMIT && !($1 =~ m/Revert\ \"/)) {

					WARN("LONG_SUMMARY_LINE",

					WARN("LONG_SUMMARY_LINE",

					     "summary line over " .

					     "summary line over " .

					     SHORTTEXT_LIMIT .

					     SHORTTEXT_LIMIT .
@@ -2351,7 +2351,7 @@ sub process { 
				# headers so we are now in the shorttext.

				# headers so we are now in the shorttext.

				$shorttext = IN_SHORTTEXT_BLANKLINE;

				$shorttext = IN_SHORTTEXT_BLANKLINE;

				$shorttext_exspc = 4;

				$shorttext_exspc = 4;

				if (length($1) > SHORTTEXT_LIMIT) {

				if (length($1) > SHORTTEXT_LIMIT && !($1 =~ m/Revert\ \"/)) {

					WARN("LONG_SUMMARY_LINE",

					WARN("LONG_SUMMARY_LINE",

					     "summary line over " .

					     "summary line over " .

					     SHORTTEXT_LIMIT .

					     SHORTTEXT_LIMIT .