glamor: Remove an extra copy of RegionNil().
authorEric Anholt <eric@anholt.net>
Fri, 27 Dec 2013 20:22:08 +0000 (12:22 -0800)
committerEric Anholt <eric@anholt.net>
Mon, 27 Jan 2014 17:30:47 +0000 (09:30 -0800)
Signed-off-by: Eric Anholt <eric@anholt.net>
Reviewed-by: Zhigang Gong <zhigang.gong@linux.intel.com>
Reviewed-by: Keith Packard <keithp@keithp.com>
glamor/glamor_compositerects.c
glamor/glamor_utils.h

index 967e228..3b6b2ed 100644 (file)
@@ -123,7 +123,7 @@ glamor_composite_rectangles(CARD8 op,
     if (!num_rects)
         return;
 
-    if (region_is_empty(dst->pCompositeClip)) {
+    if (RegionNil(dst->pCompositeClip)) {
         DEBUGF("%s: empty clip, skipping\n", __FUNCTION__);
         return;
     }
@@ -212,7 +212,7 @@ glamor_composite_rectangles(CARD8 op,
 
     if (dst->pCompositeClip->data &&
         (!pixman_region_intersect(&region, &region, dst->pCompositeClip) ||
-         region_is_empty(&region))) {
+         RegionNil(&region))) {
         DEBUGF("%s: zero-intersection between rectangles and clip\n",
                __FUNCTION__);
         pixman_region_fini(&region);
index 151a65e..ea827df 100644 (file)
@@ -815,12 +815,6 @@ glamor_translate_boxes(BoxPtr boxes, int nbox, int dx, int dy)
     }
 }
 
-static inline Bool
-region_is_empty(pixman_region16_t * region)
-{
-    return region->data && region->data->numRects == 0;
-}
-
 #ifndef ARRAY_SIZE
 #define ARRAY_SIZE(x) (sizeof(x) / sizeof(x[0]))
 #endif