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

Commit 39229466 authored by Venkata Narendra Kumar Gutta's avatar Venkata Narendra Kumar Gutta
Browse files

checkpatch: Fix the commit log false positive warnings 



Checkpatch is throwing a warning for lines that contains file delta
in the commit log.

WARNING: Possible unwrapped commit description
(prefer a maximum 75 chars per line)
kerneldir/msm-abc-xyz123.dtsi
              | 68 ++++++++++++++++++++++++++++++++++++++++----

This file delta line of the patch should be ignored for maximum characters
per line check and is taken care of already, but it's not covering if the
file has '-' in its name. Update the conditional check to cover this case.

Change-Id: If37691aa897f8a991c1643a08fbde45cc4afaaf7
Signed-off-by: default avatarVenkata Narendra Kumar Gutta <vnkgutta@codeaurora.org>
parent e59d2258

scripts/checkpatch.pl

+1 −1
Original line number Diff line number Diff line
@@ -2817,7 +2817,7 @@ sub process { 
# Check for line lengths > 75 in commit log, warn once

		if ($in_commit_log && !$commit_log_long_line &&

		    length($line) > 75 &&

		    !($line =~ /^\s*[a-zA-Z0-9_\/\.]+\s+\|\s+\d+/ ||

		    !($line =~ /^\s*[a-zA-Z0-9_\/\.\-]+\s+\|\s+\d+/ ||

					# file delta changes

		      $line =~ /^\s*(?:[\w\.\-]+\/)++[\w\.\-]+:/ ||

					# filename then :