[Mesa-dev] [PATCH 06/21] i965: Resolve GCC sign-compare warning.

Matt Turner mattst88 at gmail.com
Sun Aug 2 23:51:21 PDT 2015


On Sun, Aug 2, 2015 at 11:09 PM, Rhys Kidd <rhyskidd at gmail.com> wrote:
> mesa/src/mesa/drivers/dri/i965/intel_fbo.c: In function 'intel_validate_framebuffer':
> mesa/src/mesa/drivers/dri/i965/intel_fbo.c:734:18: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
>     for (i = 0; i < ARRAY_SIZE(fb->Attachment); i++) {
>                   ^
>
> Signed-off-by: Rhys Kidd <rhyskidd at gmail.com>
> ---
>  src/mesa/drivers/dri/i965/intel_fbo.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/intel_fbo.c b/src/mesa/drivers/dri/i965/intel_fbo.c
> index ab53603..5e4a999 100644
> --- a/src/mesa/drivers/dri/i965/intel_fbo.c
> +++ b/src/mesa/drivers/dri/i965/intel_fbo.c
> @@ -662,7 +662,7 @@ intel_validate_framebuffer(struct gl_context *ctx, struct gl_framebuffer *fb)
>     struct intel_renderbuffer *stencilRb =
>        intel_get_renderbuffer(fb, BUFFER_STENCIL);
>     struct intel_mipmap_tree *depth_mt = NULL, *stencil_mt = NULL;
> -   int i;
> +   unsigned int i;

Just use unsigned instead of "unsigned int"


More information about the mesa-dev mailing list