radeon / r200: Don't pass unused parameters to radeon_viewport
authorIan Romanick <ian.d.romanick@intel.com>
Fri, 1 Nov 2013 18:40:44 +0000 (11:40 -0700)
committerIan Romanick <ian.d.romanick@intel.com>
Tue, 5 Nov 2013 15:50:12 +0000 (07:50 -0800)
The x, y, width, and height parameters aren't used by radeon_viewport,
so don't pass them.  This should make future changes to the
dd_function_table::Viewport interface a little easier.

Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Jordan Justen <jljusten@gmail.com>
Acked-by: Alex Deucher <alexander.deucher@amd.com>
Cc: Courtney Goeltzenleuchter <courtney@lunarg.com>
src/mesa/drivers/dri/r200/r200_state.c
src/mesa/drivers/dri/radeon/radeon_common.c
src/mesa/drivers/dri/radeon/radeon_common.h
src/mesa/drivers/dri/radeon/radeon_state.c

index d3e811407bee5c690e0da235fd6a580f5381a445..ee7b9459664c6c9c650e0e88b267b050715be5f3 100644 (file)
@@ -1603,13 +1603,18 @@ void r200_vtbl_update_scissor( struct gl_context *ctx )
 static void r200Viewport( struct gl_context *ctx, GLint x, GLint y,
                            GLsizei width, GLsizei height )
 {
+   (void) x;
+   (void) y;
+   (void) width;
+   (void) height;
+
    /* Don't pipeline viewport changes, conflict with window offset
     * setting below.  Could apply deltas to rescue pipelined viewport
     * values, or keep the originals hanging around.
     */
    r200UpdateWindow( ctx );
 
-   radeon_viewport(ctx, x, y, width, height);
+   radeon_viewport(ctx);
 }
 
 static void r200DepthRange( struct gl_context *ctx, GLclampd nearval,
index 0f0945da9810e5bf3c9d65bc389283a66afb4913..7be0ba732a6154f4ea1d11597345ee65ab31460b 100644 (file)
@@ -407,7 +407,7 @@ void radeon_window_moved(radeonContextPtr radeon)
        radeonSetCliprects(radeon);
 }
 
-void radeon_viewport(struct gl_context *ctx, GLint x, GLint y, GLsizei width, GLsizei height)
+void radeon_viewport(struct gl_context *ctx)
 {
        radeonContextPtr radeon = RADEON_CONTEXT(ctx);
        __DRIcontext *driContext = radeon->dri.context;
index 636822faed2cef19282640969685e9f56df14e17..69a17278abf220487bd94993fb31c29d0820db4c 100644 (file)
@@ -23,7 +23,7 @@ void radeon_window_moved(radeonContextPtr radeon);
 void radeon_draw_buffer(struct gl_context *ctx, struct gl_framebuffer *fb);
 void radeonDrawBuffer( struct gl_context *ctx, GLenum mode );
 void radeonReadBuffer( struct gl_context *ctx, GLenum mode );
-void radeon_viewport(struct gl_context *ctx, GLint x, GLint y, GLsizei width, GLsizei height);
+void radeon_viewport(struct gl_context *ctx);
 void radeon_fbo_init(struct radeon_context *radeon);
 void
 radeon_renderbuffer_set_bo(struct radeon_renderbuffer *rb,
index 70542f68a190b8966acb2cf76c07dbdf07afef8d..625271dd16896c3f52fab7d570d20dc432224d3a 100644 (file)
@@ -1386,13 +1386,18 @@ void radeonUpdateWindow( struct gl_context *ctx )
 static void radeonViewport( struct gl_context *ctx, GLint x, GLint y,
                            GLsizei width, GLsizei height )
 {
+   (void) x;
+   (void) y;
+   (void) width;
+   (void) height;
+
    /* Don't pipeline viewport changes, conflict with window offset
     * setting below.  Could apply deltas to rescue pipelined viewport
     * values, or keep the originals hanging around.
     */
    radeonUpdateWindow( ctx );
 
-   radeon_viewport(ctx, x, y, width, height);
+   radeon_viewport(ctx);
 }
 
 static void radeonDepthRange( struct gl_context *ctx, GLclampd nearval,