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

Author: Eric Engestrom <[email protected]>
Date:   Mon Aug 15 15:51:21 2016 +0100

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 <[email protected]>
Reviewed-by: Chris Wilson <[email protected]>
Reviewed-by: Emil Velikov <[email protected]>

---

 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;
    }

_______________________________________________
mesa-commit mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-commit

Reply via email to