[Mesa-dev] [PATCH] i915: Fix GCC unused-variable warning in release build.

Timothy Arceri t_arceri at yahoo.com.au
Fri Mar 6 01:04:48 PST 2015


Reviewed-by: Timothy Arceri <t_arceri at yahoo.com.au>

On Tue, 2015-03-03 at 18:57 -0800, Vinson Lee wrote:
> i915_debug_fp.c: In function ‘i915_disassemble_program’:
> i915_debug_fp.c:302:11: warning: unused variable ‘size’ [-Wunused-variable]
>     GLuint size = program[0] & 0x1ff;
>            ^
> 
> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
> ---
>  src/mesa/drivers/dri/i915/i915_debug_fp.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/src/mesa/drivers/dri/i915/i915_debug_fp.c b/src/mesa/drivers/dri/i915/i915_debug_fp.c
> index 9b4bc76..3f09902 100644
> --- a/src/mesa/drivers/dri/i915/i915_debug_fp.c
> +++ b/src/mesa/drivers/dri/i915/i915_debug_fp.c
> @@ -299,12 +299,11 @@ print_dcl_op(GLuint opcode, const GLuint * program)
>  void
>  i915_disassemble_program(const GLuint * program, GLuint sz)
>  {
> -   GLuint size = program[0] & 0x1ff;
>     GLint i;
>  
>     printf("\t\tBEGIN\n");
>  
> -   assert(size + 2 == sz);
> +   assert(program[0] & 0x1ff + 2 == sz);
>  
>     program++;
>     for (i = 1; i < sz; i += 3, program += 3) {




More information about the mesa-dev mailing list