Re: [PATCH v5 02/10] app/test: tag tests with the test type

[David Marchand]

On Tue, Aug 15, 2023 at 5:23 PM Bruce Richardson
<bruce.richard...@intel.com> wrote:
> diff --git a/app/test/test_acl.c b/app/test/test_acl.c
> index 623f34682e..bf1466fe11 100644
> --- a/app/test/test_acl.c
> +++ b/app/test/test_acl.c
> @@ -1749,4 +1749,4 @@ test_acl(void)
>
>  #endif /* !RTE_EXEC_ENV_WINDOWS */
>
> -REGISTER_TEST_COMMAND(acl_autotest, test_acl);
> +REGISTER_FAST_TEST(acl_autotest, True, True, test_acl);

A bit surprising to read "True" and not "true" in some C code.
I see meson supports both forms.
Do you have a reason to go with "True"?
Maybe you prefer to keep it like this so it is not confused with the C
boolean keyword?


-- 
David Marchand