<div dir="ltr">Looks good.<div><br></div><div><span style="font-family:arial,sans-serif;font-size:13px">Reviewed-by: Courtney Goeltzenleuchter <<a href="mailto:courtney@lunarg.com">courtney@lunarg.com</a>></span><br>
</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, May 19, 2014 at 12:12 AM, Kenneth Graunke <span dir="ltr"><<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">These aren't necessary - all of the following code is predicated on mask<br>
being non-zero, so no code will get executed anyway.<br>
<br>
Signed-off-by: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>><br>
Cc: "10.2" <<a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.freedesktop.org</a>><br>
---<br>
src/mesa/drivers/common/meta_blit.c | 8 --------<br>
1 file changed, 8 deletions(-)<br>
<br>
diff --git a/src/mesa/drivers/common/meta_blit.c b/src/mesa/drivers/common/meta_blit.c<br>
index beb1ea5..bd6118b 100644<br>
--- a/src/mesa/drivers/common/meta_blit.c<br>
+++ b/src/mesa/drivers/common/meta_blit.c<br>
@@ -705,10 +705,6 @@ _mesa_meta_BlitFramebuffer(struct gl_context *ctx,<br>
filter, dstFlipX, dstFlipY,<br>
use_glsl_version, false)) {<br>
mask &= ~GL_COLOR_BUFFER_BIT;<br>
- if (mask == 0x0) {<br>
- _mesa_meta_end(ctx);<br>
- return;<br>
- }<br>
}<br>
}<br>
<br>
@@ -718,10 +714,6 @@ _mesa_meta_BlitFramebuffer(struct gl_context *ctx,<br>
filter, dstFlipX, dstFlipY,<br>
use_glsl_version, true)) {<br>
mask &= ~GL_DEPTH_BUFFER_BIT;<br>
- if (mask == 0x0) {<br>
- _mesa_meta_end(ctx);<br>
- return;<br>
- }<br>
}<br>
}<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
1.9.2<br>
<br>
_______________________________________________<br>
mesa-stable mailing list<br>
<a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/mesa-stable" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-stable</a><br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Courtney Goeltzenleuchter<br><div>LunarG</div><div><img src="http://media.lunarg.com/wp-content/themes/LunarG/images/logo.png" width="96" height="65"><br>
</div></div>
</div>