Commit 6dba824e authored Sep 25, 2019 by Brendan Jackman Committed by Linus Torvalds Sep 25, 2019

checkpatch: exclude sizeof sub-expressions from MACRO_ARG_REUSE

The arguments of sizeof are not evaluated so arguments are safe to re-use
in that context.  Excluding sizeof subexpressions means macros like
ARRAY_SIZE can pass checkpatch.

Link: http://lkml.kernel.org/r/20190806070833.24423-1-brendan.jackman@bluwireless.co.uk


Signed-off-by: Brendan Jackman <brendan.jackman@bluwireless.co.uk>
Acked-by: Joe Perches <joe@perches.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

parent a8dd86bf

scripts/checkpatch.pl

+1 −1

Original line number	Diff line number	Diff line
		@@ -5204,7 +5204,7 @@ sub process {
		next if ($arg =~ /\.\.\./);
		next if ($arg =~ /^type$/i);
		my $tmp_stmt = $define_stmt;
		$tmp_stmt =~ s/\b(typeof\|__typeof__\|__builtin\w+\|typecheck\s$\s$Type\s,\|\#+)\s\(\s$arg\s$\b//g;
		$tmp_stmt =~ s/\b(sizeof\|typeof\|__typeof__\|__builtin\w+\|typecheck\s$\s$Type\s,\|\#+)\s\(\s$arg\s$\b//g;
		$tmp_stmt =~ s/\#+\s*$arg\b//g;
		$tmp_stmt =~ s/\b$arg\s*\#\#//g;
		my $use_cnt = () = $tmp_stmt =~ /\b$arg\b/g;

Admin message