Mesa (master): meta: Drop unnecessary early returns in _mesa_meta_BlitFramebuffer.

Kenneth Graunke kwg at kemper.freedesktop.org
Tue May 20 00:18:00 UTC 2014


Module: Mesa
Branch: master
Commit: d89ce333ccc585ef88fe10e61de17b885e03eab4
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=d89ce333ccc585ef88fe10e61de17b885e03eab4

Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Sun May 18 19:32:44 2014 -0700

meta: Drop unnecessary early returns in _mesa_meta_BlitFramebuffer.

These aren't necessary - all of the following code is predicated on mask
being non-zero, so no code will get executed anyway.

Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
Reviewed-by: Courtney Goeltzenleuchter <courtney at lunarg.com>
Reviewed-by: Chris Forbes <chrisf at ijw.co.nz>
Cc: "10.2" <mesa-stable at lists.freedesktop.org>

---

 src/mesa/drivers/common/meta_blit.c |    8 --------
 1 file changed, 8 deletions(-)

diff --git a/src/mesa/drivers/common/meta_blit.c b/src/mesa/drivers/common/meta_blit.c
index beb1ea5..bd6118b 100644
--- a/src/mesa/drivers/common/meta_blit.c
+++ b/src/mesa/drivers/common/meta_blit.c
@@ -705,10 +705,6 @@ _mesa_meta_BlitFramebuffer(struct gl_context *ctx,
                                   filter, dstFlipX, dstFlipY,
                                   use_glsl_version, false)) {
          mask &= ~GL_COLOR_BUFFER_BIT;
-         if (mask == 0x0) {
-            _mesa_meta_end(ctx);
-            return;
-         }
       }
    }
 
@@ -718,10 +714,6 @@ _mesa_meta_BlitFramebuffer(struct gl_context *ctx,
                                   filter, dstFlipX, dstFlipY,
                                   use_glsl_version, true)) {
          mask &= ~GL_DEPTH_BUFFER_BIT;
-         if (mask == 0x0) {
-            _mesa_meta_end(ctx);
-            return;
-         }
       }
    }
 




More information about the mesa-commit mailing list