scripts/kernel-doc: fix handling of parameters with parenthesis
authorMauro Carvalho Chehab <mchehab@s-opensource.com>
Thu, 30 Mar 2017 20:11:29 +0000 (17:11 -0300)
committerJonathan Corbet <corbet@lwn.net>
Sun, 2 Apr 2017 20:06:56 +0000 (14:06 -0600)
lib/crc32c defines one parameter as:
const u32 (*tab)[256]

Better handle parenthesis, to avoid those warnings:

./lib/crc32.c:149: warning: No description found for parameter 'tab)[256]'
./lib/crc32.c:149: warning: Excess function parameter 'tab' description in 'crc32_le_generic'
./lib/crc32.c:294: warning: No description found for parameter 'tab)[256]'
./lib/crc32.c:294: warning: Excess function parameter 'tab' description in 'crc32_be_generic'

Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
scripts/kernel-doc

index a4e5cc3b38e88ebae872e7e4c560ab9fd63c0a2a..a26a5f2dce39431497bd430ab605dd061c98009c 100755 (executable)
@@ -2402,8 +2402,7 @@ sub push_parameter($$$) {
        }
 
        $anon_struct_union = 0;
-       my $param_name = $param;
-       $param_name =~ s/\[.*//;
+       $param =~ s/[\[\)].*//;
 
        if ($type eq "" && $param =~ /\.\.\.$/)
        {
@@ -2434,9 +2433,9 @@ sub push_parameter($$$) {
        # but inline preprocessor statements);
        # also ignore unnamed structs/unions;
        if (!$anon_struct_union) {
-       if (!defined $parameterdescs{$param_name} && $param_name !~ /^#/) {
+       if (!defined $parameterdescs{$param} && $param !~ /^#/) {
 
-           $parameterdescs{$param_name} = $undescribed;
+           $parameterdescs{$param} = $undescribed;
 
            if (($type eq 'function') || ($type eq 'enum')) {
                print STDERR "${file}:$.: warning: Function parameter ".