[LTP] [PATCH v2 3/3] fanotify14: fix anonymous pipe testcases
Petr Vorel
pvorel@suse.cz
Tue Mar 26 15:30:21 CET 2024
Hi Cyril,
> Hi!
> > @Cyril, @Li: How about add 2 macros with '_SIZE' in the name?
> > (TST_EXP_FAIL_ARR_SIZE and TST_EXP_FAIL2_ARR_SIZE, see diff at the end).
> Maybe just _SZ instead of full _SIZE?
> Also the TST_EXP_FAIL_ARR() is rarely used so we may as well to change
> it to include the size argument explicitly, the worst case is that user
> would have to pass down ARRAY_SIZE(errnos), which is just one more
> argument. The macro is used by a single test at the moment so it's easy
> now. Doing that would at least slown down the combinatoric explosion of
> the test macros.
Good point, I just replace ARRAY_SIZE(errnos) with size parameter in the next
version. Thanks a lot for your input.
Kind regards,
Petr
More information about the ltp
mailing list