[LTP] [PATCH v2] Rewrite statx04 test
Petr Vorel
pvorel@suse.cz
Wed Jan 19 12:45:47 CET 2022
> On 19. 01. 22 12:23, Petr Vorel wrote:
> > Hi Martin,
> >> ...
> > I'd put this into commit message (can be done during merge).
> Sure.
> >> + for (i = 0, expected_mask = 0; i < ARRAY_SIZE(attr_list); i++)
> > Before merge we should cast to prevent "comparison of integer expressions of
> > different signedness" warning:
> > for (i = 0, expected_mask = 0; i < (int)ARRAY_SIZE(attr_list); i++)
> It's better to fix that by changing the type of "i" to size_t or
+1
> unsigned int. But GCC could easily deduce that one of the values is a
> constant well within the range of signed int and not bother us with the
> warning...
Agree :).
Kind regards,
Petr
More information about the ltp
mailing list