[libcamera-devel] [PATCH] utils: gen-controls: Fix enumerators documentation

Niklas Söderlund niklas.soderlund at ragnatech.se
Sun Dec 27 11:16:59 CET 2020


Hi Laurent,

Thanks for your work.

On 2020-12-22 18:15:26 +0200, Laurent Pinchart wrote:
> The Doxygen documentation for enumerators prefixes the enumerator name
> with the enumeration name. For unscoped enumerations, this is incorrect.
> Drop the scope. This fixes warnings produced by Doxygen when multiple
> enumerators with identical names are defined in different scopes.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

Reviewed-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>

> ---
>  utils/gen-controls.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Jacopo, I believe this should fix the issue you've experienced in your
> camera sensor database series.
> 
> diff --git a/utils/gen-controls.py b/utils/gen-controls.py
> index 12a32eaaeee9..3f99b5e2ba7d 100755
> --- a/utils/gen-controls.py
> +++ b/utils/gen-controls.py
> @@ -26,7 +26,7 @@ def generate_cpp(controls):
>      enum_doc_start_template = string.Template('''/**
>   * \\enum ${name}Enum
>   * \\brief Supported ${name} values''')
> -    enum_doc_value_template = string.Template(''' * \\var ${name}Enum::${value}
> +    enum_doc_value_template = string.Template(''' * \\var ${value}
>  ${description}''')
>      doc_template = string.Template('''/**
>   * \\var ${name}
> -- 
> Regards,
> 
> Laurent Pinchart
> 
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel

-- 
Regards,
Niklas Söderlund


More information about the libcamera-devel mailing list