[PATCH 4/4] meson: Deprecate bitwise operations between different enumeration type

Stefan Klug stefan.klug at ideasonboard.com
Fri May 3 16:44:58 CEST 2024


Hi Laurent,

thanks for the patch.

On Fri, May 03, 2024 at 05:52:05AM +0300, Laurent Pinchart wrote:
> C++20 deprecates bitwise operations between different enumeration types.
> This is a bad practice even in C++17, so enable the deprecation warning.
> Doing so ensures no such operations will be added.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

Reviewed-by: Stefan Klug <stefan.klug at ideasonboard.com> 

Cheers,
Stefan

> ---
>  meson.build | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/meson.build b/meson.build
> index 39e4947f8c97..89320adbde1f 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -178,6 +178,15 @@ if cc.has_argument('-Wno-c99-designator')
>      ]
>  endif
>  
> +# Enable the C++20 deprecated enum-enum conversion warning if the compiler
> +# supports it. This helps avoiding C++20 regressions, and should be removed
> +# when switching to C++20 as the warning will then be enabled by default.
> +if cxx.has_argument('-Wdeprecated-enum-enum-conversion')
> +    cpp_arguments += [
> +        '-Wdeprecated-enum-enum-conversion',
> +    ]
> +endif
> +
>  c_arguments += common_arguments
>  cpp_arguments += common_arguments
>  
> -- 
> Regards,
> 
> Laurent Pinchart
> 


More information about the libcamera-devel mailing list