[Mesa-dev] [PATCH 03/24] gallium/aux/tgsi/tgsi_dump.c: Fix -Wsign-compare warnings

Marek Olšák maraeo at gmail.com
Fri Jun 8 04:04:34 UTC 2018


Reviewed-by: Marek Olšák <marek.olsak at amd.com>

Marek

On Tue, Jun 5, 2018 at 7:58 AM, Gert Wollny <gert.wollny at collabora.com>
wrote:

> tgsi_dump.c: In function 'iter_property':
> tgsi_dump.c:443:18: warning: comparison between signed and unsigned
> integer expressions [-Wsign-compare]
>     for (i = 0; i < prop->Property.NrTokens - 1; ++i) {
>                   ^
> tgsi_dump.c:459:13: warning: comparison between signed and unsigned
> integer expressions [-Wsign-compare]
>        if (i < prop->Property.NrTokens - 2)
>
> Signed-off-by: Gert Wollny <gert.wollny at collabora.com>
> ---
>  src/gallium/auxiliary/tgsi/tgsi_dump.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/auxiliary/tgsi/tgsi_dump.c
> b/src/gallium/auxiliary/tgsi/tgsi_dump.c
> index f6c85390e9..705d9f1cbf 100644
> --- a/src/gallium/auxiliary/tgsi/tgsi_dump.c
> +++ b/src/gallium/auxiliary/tgsi/tgsi_dump.c
> @@ -480,7 +480,7 @@ iter_property(
>     struct tgsi_iterate_context *iter,
>     struct tgsi_full_property *prop )
>  {
> -   unsigned i;
> +   int i;
>     struct dump_ctx *ctx = (struct dump_ctx *)iter;
>
>     TXT( "PROPERTY " );
> --
> 2.16.4
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20180608/f3584681/attachment.html>


More information about the mesa-dev mailing list