mirror of
https://github.com/hardkernel/linux.git
synced 2026-06-07 11:26:02 +09:00
scripts/checkpatch.pl: update to b43ae21bd1 and enable strict default
This commit is contained in:
@@ -23,7 +23,7 @@ my $tst_only;
|
||||
my $emacs = 0;
|
||||
my $terse = 0;
|
||||
my $file = 0;
|
||||
my $check = 0;
|
||||
my $check = 1;
|
||||
my $check_orig = 0;
|
||||
my $summary = 1;
|
||||
my $mailback = 0;
|
||||
@@ -3476,12 +3476,17 @@ sub process {
|
||||
}
|
||||
}
|
||||
|
||||
# unnecessary return in a void function? (a single leading tab, then return;)
|
||||
if ($sline =~ /^\+\treturn\s*;\s*$/ &&
|
||||
$prevline =~ /^\+/) {
|
||||
# unnecessary return in a void function
|
||||
# at end-of-function, with the previous line a single leading tab, then return;
|
||||
# and the line before that not a goto label target like "out:"
|
||||
if ($sline =~ /^[ \+]}\s*$/ &&
|
||||
$prevline =~ /^\+\treturn\s*;\s*$/ &&
|
||||
$linenr >= 3 &&
|
||||
$lines[$linenr - 3] =~ /^[ +]/ &&
|
||||
$lines[$linenr - 3] !~ /^[ +]\s*$Ident\s*:/) {
|
||||
WARN("RETURN_VOID",
|
||||
"void function return statements are not generally useful\n" . $herecurr);
|
||||
}
|
||||
"void function return statements are not generally useful\n" . $hereprev);
|
||||
}
|
||||
|
||||
# if statements using unnecessary parentheses - ie: if ((foo == bar))
|
||||
if ($^V && $^V ge 5.10.0 &&
|
||||
|
||||
Reference in New Issue
Block a user