diff options
author | Randy Dunlap <randy.dunlap@oracle.com> | 2007-07-19 04:48:24 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-07-19 13:04:45 -0400 |
commit | 5f8c7c98ae3888cf0a2cf320f514f75cc92f00be (patch) | |
tree | 5949c17d2d7a880838602559c01907412438f119 /scripts | |
parent | 2ac534bc127bcf31f8cb76f65b1b0b7cba5e81ac (diff) |
kernel-doc: fix unnamed struct/union warning
Fix kernel-doc warning:
Warning(linux-2.6.22-rc2-git2/include/linux/skbuff.h:316): No description found for parameter '}'
which is caused by nested anonymous structs/unions ending with:
};
};
Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
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/kernel-doc | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/scripts/kernel-doc b/scripts/kernel-doc index e5bf649e516a..08894f4aab2c 100755 --- a/scripts/kernel-doc +++ b/scripts/kernel-doc | |||
@@ -154,6 +154,7 @@ use strict; | |||
154 | 154 | ||
155 | my $errors = 0; | 155 | my $errors = 0; |
156 | my $warnings = 0; | 156 | my $warnings = 0; |
157 | my $anon_struct_union = 0; | ||
157 | 158 | ||
158 | # match expressions used to find embedded type information | 159 | # match expressions used to find embedded type information |
159 | my $type_constant = '\%([-_\w]+)'; | 160 | my $type_constant = '\%([-_\w]+)'; |
@@ -1510,8 +1511,13 @@ sub push_parameter($$$) { | |||
1510 | my $param = shift; | 1511 | my $param = shift; |
1511 | my $type = shift; | 1512 | my $type = shift; |
1512 | my $file = shift; | 1513 | my $file = shift; |
1513 | my $anon = 0; | ||
1514 | 1514 | ||
1515 | if (($anon_struct_union == 1) && ($type eq "") && | ||
1516 | ($param eq "}")) { | ||
1517 | return; # ignore the ending }; from anon. struct/union | ||
1518 | } | ||
1519 | |||
1520 | $anon_struct_union = 0; | ||
1515 | my $param_name = $param; | 1521 | my $param_name = $param; |
1516 | $param_name =~ s/\[.*//; | 1522 | $param_name =~ s/\[.*//; |
1517 | 1523 | ||
@@ -1530,16 +1536,16 @@ sub push_parameter($$$) { | |||
1530 | # handle unnamed (anonymous) union or struct: | 1536 | # handle unnamed (anonymous) union or struct: |
1531 | { | 1537 | { |
1532 | $type = $param; | 1538 | $type = $param; |
1533 | $param = "{unnamed_" . $param. "}"; | 1539 | $param = "{unnamed_" . $param . "}"; |
1534 | $parameterdescs{$param} = "anonymous\n"; | 1540 | $parameterdescs{$param} = "anonymous\n"; |
1535 | $anon = 1; | 1541 | $anon_struct_union = 1; |
1536 | } | 1542 | } |
1537 | 1543 | ||
1538 | # warn if parameter has no description | 1544 | # warn if parameter has no description |
1539 | # (but ignore ones starting with # as these are not parameters | 1545 | # (but ignore ones starting with # as these are not parameters |
1540 | # but inline preprocessor statements); | 1546 | # but inline preprocessor statements); |
1541 | # also ignore unnamed structs/unions; | 1547 | # also ignore unnamed structs/unions; |
1542 | if (!$anon) { | 1548 | if (!$anon_struct_union) { |
1543 | if (!defined $parameterdescs{$param_name} && $param_name !~ /^#/) { | 1549 | if (!defined $parameterdescs{$param_name} && $param_name !~ /^#/) { |
1544 | 1550 | ||
1545 | $parameterdescs{$param_name} = $undescribed; | 1551 | $parameterdescs{$param_name} = $undescribed; |