diff options
author | Andy Whitcroft <apw@shadowen.org> | 2008-10-16 01:02:31 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-10-16 14:21:37 -0400 |
commit | b132e5d5865325a9aa42b122c4c466903bf48348 (patch) | |
tree | 0032e6513e7ae5353357c0fbf605ecda6a9cf831 /scripts | |
parent | 170d3a22688f06fda42e353bbccd0f3df89f3d94 (diff) |
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>
Diffstat (limited to 'scripts')
-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 { | |||
2155 | if ($dstat ne '' && | 2155 | if ($dstat ne '' && |
2156 | $dstat !~ /^(?:$Ident|-?$Constant)$/ && | 2156 | $dstat !~ /^(?:$Ident|-?$Constant)$/ && |
2157 | $dstat !~ /$exceptions/ && | 2157 | $dstat !~ /$exceptions/ && |
2158 | $dstat !~ /^\.$Ident\s*=/ && | ||
2158 | $dstat =~ /$Operators/) | 2159 | $dstat =~ /$Operators/) |
2159 | { | 2160 | { |
2160 | ERROR("Macros with complex values should be enclosed in parenthesis\n" . "$here\n$ctx\n"); | 2161 | ERROR("Macros with complex values should be enclosed in parenthesis\n" . "$here\n$ctx\n"); |