From 8af1b540c52fbc9fe34f1b2fef1f728b205ab630 Mon Sep 17 00:00:00 2001 From: Eric Engestrom Date: Mon, 15 Aug 2016 15:51:21 +0100 Subject: [PATCH] i915: remove unnecessary `if` if (x) return true; else return false; can be simplified as: return x; since `x` is already a boolean expression. Signed-off-by: Eric Engestrom Reviewed-by: Chris Wilson Reviewed-by: Emil Velikov --- src/mesa/drivers/dri/i915/intel_screen.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/mesa/drivers/dri/i915/intel_screen.c b/src/mesa/drivers/dri/i915/intel_screen.c index 77f03fa..19f7a6a 100644 --- a/src/mesa/drivers/dri/i915/intel_screen.c +++ b/src/mesa/drivers/dri/i915/intel_screen.c @@ -489,9 +489,7 @@ intel_query_image(__DRIimage *image, int attrib, int *value) *value = image->planar_format->components; return true; case __DRI_IMAGE_ATTRIB_FD: - if (drm_intel_bo_gem_export_to_prime(image->region->bo, value) == 0) - return true; - return false; + return !drm_intel_bo_gem_export_to_prime(image->region->bo, value); default: return false; } -- 2.7.4