On Wed, Aug 16, 2023 at 04:56:23PM +0200, David Marchand wrote: > 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? > This info is pulled out by python script, so I believe it is capitalized so I can process them as boolean values there. However, looking at the script, it doesn't actually ever convert to boolean right now, so this may be a holdover from earlier versions. When we pass the values to meson, they are all forced to lowercase.
/Bruce
