elm_entry: modify to not allocate Eina_Rectangle
authorMinwoo, Lee <minwoo47.lee@samsung.net>
Mon, 22 Jun 2015 09:47:22 +0000 (11:47 +0200)
committerCedric BAIL <cedric@osg.samsung.com>
Mon, 22 Jun 2015 10:36:57 +0000 (12:36 +0200)
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>
src/lib/elm_entry.c

index 6b8002b..1724cb6 100644 (file)
@@ -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);
      }