diff options
author | Ian Romanick <[email protected]> | 2010-03-03 17:17:56 -0800 |
---|---|---|
committer | Ian Romanick <[email protected]> | 2010-03-05 16:04:59 -0800 |
commit | 66c0abba6ba89d326555389d9539894402303286 (patch) | |
tree | 101127ccf3b5f263675fe8c77da98fd167b1e99b /src/mesa/swrast/s_accum.c | |
parent | ebbec7e4e44fccf496a9a663bdcc643efead6e95 (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 <[email protected]>
Diffstat (limited to 'src/mesa/swrast/s_accum.c')
-rw-r--r-- | src/mesa/swrast/s_accum.c | 6 |
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; |