On Wed, Sep 9, 2015 at 7:18 PM, Ian Romanick <i...@freedesktop.org> wrote:
> From: Ian Romanick <ian.d.roman...@intel.com>
>
> intel_fbo.c: In function 'intel_alloc_window_storage':
> intel_fbo.c:415:48: warning: unused parameter 'ctx' [-Wunused-parameter]
> intel_alloc_window_storage(struct gl_context * ctx, struct
> gl_renderbuffer *rb,
> ^
> intel_fbo.c: In function 'intel_nop_alloc_storage':
> intel_fbo.c:428:74: warning: unused parameter 'rb' [-Wunused-parameter]
> intel_nop_alloc_storage(struct gl_context * ctx, struct gl_renderbuffer
> *rb,
> ^
> intel_fbo.c:429:32: warning: unused parameter 'internalFormat'
> [-Wunused-parameter]
> GLenum internalFormat, GLuint width, GLuint
> height)
> ^
> intel_fbo.c:429:55: warning: unused parameter 'width' [-Wunused-parameter]
> GLenum internalFormat, GLuint width, GLuint
> height)
> ^
> intel_fbo.c:429:69: warning: unused parameter 'height' [-Wunused-parameter]
> GLenum internalFormat, GLuint width, GLuint
> height)
> ^
> intel_fbo.c: In function 'intel_blit_framebuffer_with_blitter':
> intel_fbo.c:790:61: warning: unused parameter 'filter' [-Wunused-parameter]
> GLbitfield mask, GLenum filter)
> ^
>
> Signed-off-by: Ian Romanick <ian.d.roman...@intel.com>
> ---
> src/mesa/drivers/dri/i965/intel_fbo.c | 11 ++++++++---
> 1 file changed, 8 insertions(+), 3 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/intel_fbo.c
> b/src/mesa/drivers/dri/i965/intel_fbo.c
> index 64d57e8..cc90a49 100644
> --- a/src/mesa/drivers/dri/i965/intel_fbo.c
> +++ b/src/mesa/drivers/dri/i965/intel_fbo.c
> @@ -415,6 +415,7 @@ static GLboolean
> intel_alloc_window_storage(struct gl_context * ctx, struct
> gl_renderbuffer *rb,
> GLenum internalFormat, GLuint width, GLuint
> height)
> {
> + (void) ctx;
> assert(rb->Name == 0);
> rb->Width = width;
> rb->Height = height;
> @@ -428,6 +429,10 @@ static GLboolean
> intel_nop_alloc_storage(struct gl_context * ctx, struct gl_renderbuffer
> *rb,
> GLenum internalFormat, GLuint width, GLuint
> height)
> {
> + (void) rb;
> + (void) internalFormat;
> + (void) width;
> + (void) height;
>
Using a macro might make the code more readable.
#define UNUSED(x) (void) (x)
_mesa_problem(ctx, "intel_nop_alloc_storage should never be called.");
> return false;
> }
> @@ -787,7 +792,7 @@ intel_blit_framebuffer_with_blitter(struct gl_context
> *ctx,
> GLint srcX1, GLint srcY1,
> GLint dstX0, GLint dstY0,
> GLint dstX1, GLint dstY1,
> - GLbitfield mask, GLenum filter)
> + GLbitfield mask)
> {
> struct brw_context *brw = brw_context(ctx);
>
> @@ -907,7 +912,7 @@ intel_blit_framebuffer(struct gl_context *ctx,
> mask = intel_blit_framebuffer_with_blitter(ctx, readFb, drawFb,
> srcX0, srcY0, srcX1, srcY1,
> dstX0, dstY0, dstX1, dstY1,
> - mask, filter);
> + mask);
> if (mask == 0x0)
> return;
>
> @@ -945,7 +950,7 @@ gen4_blit_framebuffer(struct gl_context *ctx,
> mask = intel_blit_framebuffer_with_blitter(ctx, readFb, drawFb,
> srcX0, srcY0, srcX1, srcY1,
> dstX0, dstY0, dstX1, dstY1,
> - mask, filter);
> + mask);
> if (mask == 0x0)
> return;
>
> --
> 2.1.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
Irrespective of the above suggestion, this patch is:
Reviewed-by: Anuj Phogat <anuj.pho...@gmail.com>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
