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

Commit fd1b57ac authored by Andy Whitcroft's avatar Andy Whitcroft Committed by Linus Torvalds
Browse files

checkpatch: high precedence operators do not require additional parentheses in #defines 



With any very high precedence operator it is not necessary to enforce
additional parentheses around simple negated expressions.  This prevents
us requesting further perentheses around the following:

    #define PMEM_IS_FREE(id, index) !(pmem[id].bitmap[index].allocated)

For now add logical and bitwise not and unary minus.

Signed-off-by: default avatarAndy Whitcroft <apw@canonical.com>
Cc: Joe Perches <joe@perches.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent e45bab8e

scripts/checkpatch.pl

+1 −1
Original line number Diff line number Diff line
@@ -2916,7 +2916,7 @@ sub process { 
			if ($dstat ne '' &&

			    $dstat !~ /^(?:$Ident|-?$Constant),$/ &&			# 10, // foo(),

			    $dstat !~ /^(?:$Ident|-?$Constant);$/ &&			# foo();

			    $dstat !~ /^(?:$Ident|-?$Constant)$/ &&			# 10 // foo()

			    $dstat !~ /^[!~-]?(?:$Ident|$Constant)$/ &&		# 10 // foo() // !foo // ~foo // -foo

			    $dstat !~ /^'X'$/ &&					# character constants

			    $dstat !~ /$exceptions/ &&

			    $dstat !~ /^\.$Ident\s*=/ &&				# .foo =