From 9943b6612b875286ab183c17e6ba89e2eafc77ab Mon Sep 17 00:00:00 2001 From: Vinson Lee Date: Sun, 3 Nov 2013 14:42:18 -0800 Subject: [PATCH] i915: Fix logic_op check. Fixes "Macro compares unsigned to 0" defect reported by Coverity. Signed-off-by: Vinson Lee Reviewed-by: Ian Romanick --- src/mesa/drivers/dri/i915/intel_blit.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i915/intel_blit.c b/src/mesa/drivers/dri/i915/intel_blit.c index 7b59708343b..5147d8ee307 100644 --- a/src/mesa/drivers/dri/i915/intel_blit.c +++ b/src/mesa/drivers/dri/i915/intel_blit.c @@ -534,8 +534,7 @@ intelEmitImmediateColorExpandBlit(struct intel_context *intel, return false; } - assert( logic_op - GL_CLEAR >= 0 ); - assert( logic_op - GL_CLEAR < 0x10 ); + assert((logic_op >= GL_CLEAR) && (logic_op <= (GL_CLEAR + 0x0f))); assert(dst_pitch > 0); if (w < 0 || h < 0) -- 2.30.2