Robert Millan <rmh <at> aybabtu.com> writes:

> Fixes a build warning in fnmatch_loop.c.
> 
> 
> -    : p - startp + 1);      \
> +    : (unsigned) (p - startp) + 1);      \ 

I'm not a fan of unnecessary casts.  Can't we instead write this as:

: p - startp + 1U;

to still show that we intend for unsigned math, but without a cast?

-- 
Eric Blake




Reply via email to