Hi Sakari,
On 1/27/24 10:53, Sakari Ailus wrote:
> In a rather unusual arrangement in include/media/v4l2-vp9.h struct
> v4l2_vp9_frame_symbol_counts has fields that are arrays of pointers, not a
> pointer to an array, which is what's usually done.
>
Yet another kernel-doc bug. I have a list of 5 or 6 or 8 bugs that are
similar to this one, but I didn't have this one.
> Add support for such arrays of pointers to kernel-doc.
>
> Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx>
Did this work for you?
I still see the same warnings as before this patch is applied.
> ---
> Hi Ricardo,
>
> I tried to ram it together with processing the rest but it seemed
> to complicate processing the usual fields.
>
> scripts/kernel-doc | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/scripts/kernel-doc b/scripts/kernel-doc
> index e8aefd258a29..728b1ffc0478 100755
> --- a/scripts/kernel-doc
> +++ b/scripts/kernel-doc
> @@ -1509,6 +1509,15 @@ sub create_parameterlist($$$$) {
> $type =~ s/([^\(]+\(\*?)\s*$param/$1/;
> save_struct_actual($param);
> push_parameter($param, $type, $arg, $file, $declaration_name);
> + } elsif ($arg =~ m/\(.+\)\s*\[/) {
> + # array-of-pointers
> + $arg =~ tr/#/,/;
> + $arg =~ m/[^\(]+\(\s*\*\s*([\w\[\]\.]*?)\s*(\s*\[\s*[\w]+\s*\]\s*)*\)/;
> + $param = $1;
> + $type = $arg;
> + $type =~ s/([^\(]+\(\*?)\s*$param/$1/;
> + save_struct_actual($param);
> + push_parameter($param, $type, $arg, $file, $declaration_name);
> } elsif ($arg) {
> $arg =~ s/\s*:\s*/:/g;
> $arg =~ s/\s*\[/\[/g;
Thanks.
--
#Randy
