From: Søren Sandmann Pedersen <ssp@redhat.com>
Date: Tue, 2 Aug 2011 07:03:48 +0000 (-0400)
Subject: Use find_box_for_y() in pixman_region_contains_point() too
X-Git-Tag: 1.0_branch~209
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=e5d85ce6629c84b9dad5a9c76bd9f895157c5a74;p=profile%2Fivi%2Fpixman.git

Use find_box_for_y() in pixman_region_contains_point() too

The same binary search from the previous commit can be used in this
function too.

V2: Remove check from loop that is not needed anymore, pointed out by
Andrea Canciani.
---

diff --git a/pixman/pixman-region.c b/pixman/pixman-region.c
index 71995fd..9ff5157 100644
--- a/pixman/pixman-region.c
+++ b/pixman/pixman-region.c
@@ -2378,13 +2378,13 @@ PREFIX (_contains_point) (region_type_t * region,
         return(TRUE);
     }
 
-    for (pbox = PIXREGION_BOXPTR (region), pbox_end = pbox + numRects;
-	 pbox != pbox_end;
-	 pbox++)
-    {
-        if (y >= pbox->y2)
-	    continue;           /* not there yet */
+    pbox = PIXREGION_BOXPTR (region);
+    pbox_end = pbox + numRects;
+
+    pbox = find_box_for_y (pbox, pbox_end, y);
 
+    for (;pbox != pbox_end; pbox++)
+    {
         if ((y < pbox->y1) || (x < pbox->x1))
 	    break;              /* missed it */