HAOUAS Elyes has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/61467 )
Change subject: util/lint/checkpatch.pl: Remove unneeded whitespaces and fix some typos ......................................................................
util/lint/checkpatch.pl: Remove unneeded whitespaces and fix some typos
This is to reduce difference with linux v5.16.
Change-Id: I4aa7abce83b41ccd5129717cd3bf85be19ec4807 Signed-off-by: Elyes HAOUAS ehaouas@noos.fr --- M util/lint/checkpatch.pl 1 file changed, 5 insertions(+), 5 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/67/61467/1
diff --git a/util/lint/checkpatch.pl b/util/lint/checkpatch.pl index 01e55b5..a6a595c 100755 --- a/util/lint/checkpatch.pl +++ b/util/lint/checkpatch.pl @@ -4718,7 +4718,7 @@ $lines[$linenr - 3] !~ /^[ +]\s*$Ident\s*:/) { WARN("RETURN_VOID", "void function return statements are not generally useful\n" . $hereprev); - } + }
# if statements using unnecessary parentheses - ie: if ((foo == bar)) if ($perl_version_ok && @@ -4819,7 +4819,7 @@ # conditional. substr($s, 0, length($c), ''); $s =~ s/\n.*//g; - $s =~ s/$;//g; # Remove any comments + $s =~ s/$;//g; # Remove any comments if (length($c) && $s !~ /^\s*{?\s*\*\s*$/ && $c !~ /}\s*while\s*/) { @@ -4858,7 +4858,7 @@ # if and else should not have general statements after it if ($line =~ /^.\s*(?:}\s*)?else\b(.*)/) { my $s = $1; - $s =~ s/$;//g; # Remove any comments + $s =~ s/$;//g; # Remove any comments if ($s !~ /^\s*(?:\sif|(?:{|)\s*\?\s*$)/) { ERROR("TRAILING_STATEMENTS", "trailing statements should be on next line\n" . $herecurr); @@ -5043,7 +5043,7 @@ { }
- # Flatten any obvious string concatentation. + # Flatten any obvious string concatenation. while ($dstat =~ s/($String)\s*$Ident/$1/ || $dstat =~ s/$Ident\s*($String)/$1/) { @@ -6545,7 +6545,7 @@ exit(0); }
- # This is not a patch, and we are are in 'no-patch' mode so + # This is not a patch, and we are in 'no-patch' mode so # just keep quiet. if (!$chk_patch && !$is_patch) { exit(0);