[Mesa-dev] [PATCH 4/4] mesa: s/%u/%d/ in _mesa_error() call in check_layer()

Marek Olšák maraeo at gmail.com
Tue Dec 5 18:47:09 UTC 2017


For the series:

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

Marek

On Tue, Dec 5, 2017 at 6:00 PM, Brian Paul <brianp at vmware.com> wrote:
> The layer parameter is signed.  Fixes the error message seen when
> running the arb_texture_multisample-errors test which checks a
> negative layer value.
> ---
>  src/mesa/main/fbobject.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/src/mesa/main/fbobject.c b/src/mesa/main/fbobject.c
> index f7702f1..30287ab 100644
> --- a/src/mesa/main/fbobject.c
> +++ b/src/mesa/main/fbobject.c
> @@ -3189,8 +3189,7 @@ check_layer(struct gl_context *ctx, GLenum target, GLint layer,
>      *     and layer is negative."
>      */
>     if (layer < 0) {
> -      _mesa_error(ctx, GL_INVALID_VALUE,
> -                  "%s(layer %u < 0)", caller, layer);
> +      _mesa_error(ctx, GL_INVALID_VALUE, "%s(layer %d < 0)", caller, layer);
>        return false;
>     }
>
> --
> 1.9.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list