On 09/03/13 17:34, Joe Perches wrote:
> Many kernel-doc return description sections headers
> use variants of the "Return:" section prefix.
>
> (some or maybe even most of these aren't in
> kernel-doc sections, but many are)
>
> $ git grep -E -i "^\s*\*\s*return[s]?:"| \
> cut -f2- -d":" | awk '{print $1 $2}' | \
> sort | uniq -c
> 121 *return:
> 838 *Return:
> 778 *RETURN:
> 191 *returns:
> 1965 *Returns:
> 603 *RETURNS:
>
> Rather than change all of the variants to the
> canonical "Return:", adapt the script to accept
> case insensitive "Return:" and "Returns:"
>
> Signed-off-by: Joe Perches <[email protected]>
> ---
>
> I don't know that this actually works.
> It does seem right though.
>
> Randy? Rob? Can you test this please?
Works for me. Thanks.
Tested-by: Randy Dunlap <[email protected]>
>
> scripts/kernel-doc | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/scripts/kernel-doc b/scripts/kernel-doc
> index 4305b2f..2b96411 100755
> --- a/scripts/kernel-doc
> +++ b/scripts/kernel-doc
> @@ -418,6 +418,7 @@ sub dump_section {
> $parameterdescs{$name} = $contents;
> $sectcheck = $sectcheck . $name . " ";
> } else {
> + $name = "Return" if $name =~ /^return[s]?$/i;
> # print STDERR "other section '$name' = '$contents'\n";
> if (defined($sections{$name}) && ($sections{$name} ne "")) {
> print STDERR "Error(${file}:$.): duplicate section name
> '$name'\n";
>
>
> --
--
~Randy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
