From: Minwoo, Lee <minwoo47.lee@samsung.net>
Date: Mon, 22 Jun 2015 09:47:22 +0000 (+0200)
Subject: elm_entry: modify to not allocate Eina_Rectangle
X-Git-Tag: v1.15.0-alpha1~161
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=1886838fb47940bcc3e90c855c71024323e7dd9e;p=platform%2Fupstream%2Felementary.git

elm_entry: modify to not allocate Eina_Rectangle

Summary: modify to not allocate Eina_Rectangle and use it directly on the stack with EINA_RECTANGLE_SET

Reviewers: raster, woohyun, cedric

Reviewed By: cedric

Differential Revision: https://phab.enlightenment.org/D2737

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
---

diff --git a/src/lib/elm_entry.c b/src/lib/elm_entry.c
index 6b8002b..1724cb6 100644
--- a/src/lib/elm_entry.c
+++ b/src/lib/elm_entry.c
@@ -439,15 +439,14 @@ _viewport_region_get(Evas_Object *obj)
      {
         if (eo_isa(parent, ELM_INTERFACE_SCROLLABLE_MIXIN))
           {
-             Eina_Rectangle *pr = eina_rectangle_new(0, 0, 0, 0);
-             evas_object_geometry_get(parent, &pr->x, &pr->y, &pr->w, &pr->h);
-             if (!eina_rectangle_intersection(rect, pr))
+             Eina_Rectangle r;
+             EINA_RECTANGLE_SET(&r, 0, 0, 0, 0);
+             evas_object_geometry_get(parent, &r.x, &r.y, &r.w, &r.h);
+             if (!eina_rectangle_intersection(rect, &r))
                {
                   rect->x = rect->y = rect->w = rect->h = 0;
-                  eina_rectangle_free(pr);
                   break;
                }
-             eina_rectangle_free(pr);
           }
         parent = elm_widget_parent_get(parent);
      }