[libcamera-devel] [PATCH v1 1/4] test: v4l2_compat: Reduce indentation

[Paul Elder]

On Thu, Dec 22, 2022 at 03:01:29AM +0200, Laurent Pinchart via libcamera-devel wrote:
> Return early with subdir_done() to reduce indentation in case the
> v4l2_compat layer is not enabled. This matches our usual code patterns
> in meson.build files, and prepares for enabling the v4l2_compat test
> with ASan.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

Reviewed-by: Paul Elder <paul.elder at ideasonboard.com>

> ---
>  test/v4l2_compat/meson.build | 16 +++++++++-------
>  1 file changed, 9 insertions(+), 7 deletions(-)
> 
> diff --git a/test/v4l2_compat/meson.build b/test/v4l2_compat/meson.build
> index 878095892679..10c4675286b3 100644
> --- a/test/v4l2_compat/meson.build
> +++ b/test/v4l2_compat/meson.build
> @@ -1,5 +1,9 @@
>  # SPDX-License-Identifier: CC0-1.0
>  
> +if not is_variable('v4l2_compat')
> +    subdir_done()
> +endif
> +
>  # If ASan is enabled, the link order runtime check will fail as v4l2-ctl and
>  # v4l2-compliance are not linked to ASan. Skip the test in that case.
>  #
> @@ -10,11 +14,9 @@ if get_option('b_sanitize').contains('address')
>      subdir_done()
>  endif
>  
> -if is_variable('v4l2_compat')
> -    v4l2_compat_test = files('v4l2_compat_test.py')
> +v4l2_compat_test = files('v4l2_compat_test.py')
>  
> -    test('v4l2_compat_test', v4l2_compat_test,
> -         args : v4l2_compat,
> -         suite : 'v4l2_compat',
> -         timeout : 60)
> -endif
> +test('v4l2_compat_test', v4l2_compat_test,
> +     args : v4l2_compat,
> +     suite : 'v4l2_compat',
> +     timeout : 60)