void foo(int a) switch (a) { case 'h': fun1(); exit(1); default: }
creates a warning Possible switch case/default not preceded by break or fallthrough comment exit( should be treated like return. Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de> --- v2: Allow whitespace between 'exit' and '('. --- scripts/checkpatch.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 2287a0bca8..690fe07d1b 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -6084,7 +6084,7 @@ sub process { next if ($fline =~ /^.[\s$;]*$/); $has_statement = 1; $count++; - $has_break = 1 if ($fline =~ /\bswitch\b|\b(?:break\s*;[\s$;]*$|return\b|goto\b|continue\b)/); + $has_break = 1 if ($fline =~ /\bswitch\b|\b(?:break\s*;[\s$;]*$|exit\s*\(\b|return\b|goto\b|continue\b)/); } if (!$has_break && $has_statement) { WARN("MISSING_BREAK", -- 2.11.0