diff options
| author | Andy Whitcroft <apw@shadowen.org> | 2008-10-16 07:02:31 +0200 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-10-16 20:21:37 +0200 |
| commit | b132e5d5865325a9aa42b122c4c466903bf48348 (patch) | |
| tree | 0032e6513e7ae5353357c0fbf605ecda6a9cf831 | |
| parent | checkpatch: handle do without braces if we have enough context (diff) | |
| download | linux-b132e5d5865325a9aa42b122c4c466903bf48348.tar.xz linux-b132e5d5865325a9aa42b122c4c466903bf48348.zip | |
checkpatch: macros which define structure members are not complex
We often see macros which define structure members, these are not complex
and necessarily do not have braces or brackets. For example:
#define _PLIST_HEAD_INIT(head) \
.prio_list = LIST_HEAD_INIT((head).prio_list), \
.node_list = LIST_HEAD_INIT((head).node_list)
Signed-off-by: Andy Whitcroft <apw@shadowen.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
| -rwxr-xr-x | scripts/checkpatch.pl | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 19690a2ddb74..6eceda7a4bd9 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -2155,6 +2155,7 @@ sub process { if ($dstat ne '' && $dstat !~ /^(?:$Ident|-?$Constant)$/ && $dstat !~ /$exceptions/ && + $dstat !~ /^\.$Ident\s*=/ && $dstat =~ /$Operators/) { ERROR("Macros with complex values should be enclosed in parenthesis\n" . "$here\n$ctx\n"); |