Commit f9b5c530 authored Mar 30, 2017 by Mauro Carvalho Chehab Committed by Jonathan Corbet Apr 02, 2017

scripts/kernel-doc: fix handling of parameters with parenthesis



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>

parent b97f193a

scripts/kernel-doc

+3 −4

Original line number	Diff line number	Diff line
		@@ -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 ".

Admin message