From da54d2e576426122009be083ecbfb9eefd8a3799 Mon Sep 17 00:00:00 2001 From: Paul Berry Date: Fri, 13 Jul 2012 18:30:55 -0700 Subject: [PATCH] i965/blorp: Simplify check that src/dst width/height match. When checking that the source and destination dimensions match, we don't need to store the width and height in variables; doing so just risks confusion since right after the check, we do clipping and scissoring, which may alter the width and height. No functional change. Reviewed-by: Chad Versace --- src/mesa/drivers/dri/i965/brw_blorp_blit.cpp | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_blorp_blit.cpp b/src/mesa/drivers/dri/i965/brw_blorp_blit.cpp index f72145fe04e..eb7935969da 100644 --- a/src/mesa/drivers/dri/i965/brw_blorp_blit.cpp +++ b/src/mesa/drivers/dri/i965/brw_blorp_blit.cpp @@ -191,10 +191,8 @@ try_blorp_blit(struct intel_context *intel, fixup_mirroring(mirror_y, dstY0, dstY1); /* Make sure width and height match */ - GLsizei width = srcX1 - srcX0; - GLsizei height = srcY1 - srcY0; - if (width != dstX1 - dstX0) return false; - if (height != dstY1 - dstY0) return false; + if (srcX1 - srcX0 != dstX1 - dstX0) return false; + if (srcY1 - srcY0 != dstY1 - dstY0) return false; /* If the destination rectangle needs to be clipped or scissored, do so. */ -- 2.30.2