[libcamera-devel] [PATCH v3 05/14] libcamera: controls: Rename enumerate values

Laurent Pinchart laurent.pinchart at ideasonboard.com
Thu Oct 22 04:43:42 CEST 2020


Hi Jacopo,

Thank you for the patch.

On Wed, Oct 21, 2020 at 04:36:26PM +0200, Jacopo Mondi wrote:
> Rename the enumeration of supported values with the suffix "Enum"
> in place of "Values" to prepare to re-use the suffix "Values"
> for the vector of Control's value introduced by the next patch.

Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

> Signed-off-by: Jacopo Mondi <jacopo at jmondi.org>
> ---
>  utils/gen-controls.py | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/utils/gen-controls.py b/utils/gen-controls.py
> index 93cb3885c3da..bf681503f86a 100755
> --- a/utils/gen-controls.py
> +++ b/utils/gen-controls.py
> @@ -24,9 +24,9 @@ def format_description(description):
>  
>  def generate_cpp(controls):
>      enum_doc_start_template = string.Template('''/**
> - * \\enum ${name}Values
> + * \\enum ${name}Enum
>   * \\brief Supported ${name} values''')
> -    enum_doc_value_template = string.Template(''' * \\var ${name}Values::${value}
> +    enum_doc_value_template = string.Template(''' * \\var ${name}Enum::${value}
>  ${description}''')
>      doc_template = string.Template('''/**
>   * \\var ${name}
> @@ -98,7 +98,7 @@ ${description}
>  
>  
>  def generate_h(controls):
> -    enum_template_start = string.Template('''enum ${name}Values {''')
> +    enum_template_start = string.Template('''enum ${name}Enum {''')
>      enum_value_template = string.Template('''\t${name} = ${value},''')
>      template = string.Template('''extern const Control<${type}> ${name};''')
>  

-- 
Regards,

Laurent Pinchart


More information about the libcamera-devel mailing list