summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorIan Romanick <ian.d.romanick@intel.com>2010-03-03 17:17:56 -0800
committerIan Romanick <ian.d.romanick@intel.com>2010-03-05 16:04:59 -0800
commit66c0abba6ba89d326555389d9539894402303286 (patch)
tree101127ccf3b5f263675fe8c77da98fd167b1e99b
parentebbec7e4e44fccf496a9a663bdcc643efead6e95 (diff)
swrast: Remove redundant test of the visual's accumulation buffer size
If the visual doesn't have an accumulation buffer, the renderbuffer passed into _swrast_clear_accum_buffer will be NULL anyway. There is no reason the check the visual. Moreover, the test erroneously checks the context's visual instead of the visual of the current DrawBuffer. With FBOs these may be different. Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
-rw-r--r--src/mesa/swrast/s_accum.c6
1 files changed, 1 insertions, 5 deletions
diff --git a/src/mesa/swrast/s_accum.c b/src/mesa/swrast/s_accum.c
index 2dd9ca6348b..854e106b7f0 100644
--- a/src/mesa/swrast/s_accum.c
+++ b/src/mesa/swrast/s_accum.c
@@ -130,11 +130,7 @@ _swrast_clear_accum_buffer( GLcontext *ctx, struct gl_renderbuffer *rb )
SWcontext *swrast = SWRAST_CONTEXT(ctx);
GLuint x, y, width, height;
- if (ctx->Visual.accumRedBits == 0) {
- /* No accumulation buffer! Not an error. */
- return;
- }
-
+ /* No accumulation buffer! Not an error. */
if (!rb || !rb->Data)
return;