Reviewed-by: Jordan Justen <[email protected]> and committed.
On 2015-09-30 06:18:52, Rhys Kidd wrote: > Commit 1665d29ee3125743fd6daf3c43fc715f543d5669 introduced an incorrect > format specifier that operates on GLintptr indirect within the function > _mesa_DispatchComputeIndirect(). > > This patch mitigates the introduced GCC warning: > > src/mesa/main/compute.c: In function '_mesa_DispatchComputeIndirect': > src/mesa/main/compute.c:53:7: warning: format '%d' expects argument of type > 'int', but argument 3 has type 'GLintptr' [-Wformat=] > _mesa_debug(ctx, "glDispatchComputeIndirect(%d)\n", indirect); > ^ > > v2: Amend for Boyan Ding <[email protected]> feedback. > Signed-off-by: Rhys Kidd <[email protected]> > --- > src/mesa/main/compute.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/mesa/main/compute.c b/src/mesa/main/compute.c > index 8bc3bcd..53e7a50 100644 > --- a/src/mesa/main/compute.c > +++ b/src/mesa/main/compute.c > @@ -50,7 +50,7 @@ _mesa_DispatchComputeIndirect(GLintptr indirect) > GET_CURRENT_CONTEXT(ctx); > > if (MESA_VERBOSE & VERBOSE_API) > - _mesa_debug(ctx, "glDispatchComputeIndirect(%d)\n", indirect); > + _mesa_debug(ctx, "glDispatchComputeIndirect(%ld)\n", (long) indirect); > > if (!_mesa_validate_DispatchComputeIndirect(ctx, indirect)) > return; > -- > 2.1.4 > _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
