On 2019-10-12 14:28, Martin Wilck wrote:
> -     if (snprintf(buf, bufsiz, "%s%s%d", mapname, delim, part) >= bufsiz)
> +     if (snprintf(buf, bufsiz, "%s%s%d", mapname, delim, part)
> +         >= (int)bufsiz)
>               return 0;

Please don't insert casts like this. I think enabling -Wsign-compare is
wrong because it makes the source code ugly.

Thank,

Bart.

--
dm-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/dm-devel

Reply via email to