elementary: efl_ui_view_list cleanup
authorFelipe Magno de Almeida <felipe@expertisesolutions.com.br>
Wed, 1 Aug 2018 22:45:36 +0000 (19:45 -0300)
committerYeongjong Lee <yj34.lee@samsung.com>
Wed, 8 Aug 2018 01:54:09 +0000 (10:54 +0900)
Summary:
private data cleanup
removed callbacks

Reviewers: felipealmeida, SanghyeonLee

Reviewed By: felipealmeida

Subscribers: Hermet

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

Change-Id: I20c7af06992ba7b03ab72ff773c6f7a866e0f75e

src/lib/elementary/efl_ui_view_list.c
src/lib/elementary/efl_ui_view_list_private.h

index 61f4c4e..09562be 100644 (file)
@@ -2,13 +2,10 @@
 # include "elementary_config.h"
 #endif
 #define EFL_ACCESS_OBJECT_PROTECTED
-#define EFL_ACCESS_WIDGET_ACTION_PROTECTED
 #define EFL_ACCESS_SELECTION_PROTECTED
 #define EFL_UI_SCROLL_MANAGER_PROTECTED
 #define EFL_UI_SCROLLBAR_PROTECTED
 #define EFL_UI_SCROLLBAR_BETA
-#define EFL_GFX_SIZE_HINT_PROTECTED
-#define EFL_UI_VIEW_LIST_PROTECTED
 #define EFL_UI_FOCUS_COMPOSITION_PROTECTED
 #define EFL_UI_WIDGET_FOCUS_MANAGER_PROTECTED
 
@@ -400,12 +397,6 @@ _efl_ui_view_list_hbar_unpress_cb(void *data,
 }
 
 static void
-_scroll_cb(void *data EINA_UNUSED, const Efl_Event *event EINA_UNUSED)
-{
-   //scroll cb
-}
-
-static void
 _efl_ui_view_list_bar_size_changed_cb(void *data, const Efl_Event *event EINA_UNUSED)
 {
    Eo *obj = data;
@@ -490,8 +481,8 @@ _efl_ui_view_list_edje_object_attach(Eo *obj)
      (obj, "reload", "efl", _efl_ui_view_list_reload_cb, obj);
   //Vertical bar
    efl_layout_signal_callback_add
-     (obj, "drag", "efl.dragable.vbar", _efl_ui_view_list_vbar_drag_cb,
-     obj);
+     (obj, "drag", "efl.dragable.vbar",
+     _efl_ui_view_list_vbar_drag_cb, obj);
    efl_layout_signal_callback_add
      (obj, "drag,set", "efl.dragable.vbar",
      _efl_ui_view_list_edje_drag_cb, obj);
@@ -516,8 +507,8 @@ _efl_ui_view_list_edje_object_attach(Eo *obj)
 
   //Horizontal bar
    efl_layout_signal_callback_add
-     (obj, "drag", "efl.dragable.hbar", _efl_ui_view_list_hbar_drag_cb,
-     obj);
+     (obj, "drag", "efl.dragable.hbar",
+     _efl_ui_view_list_hbar_drag_cb, obj);
    efl_layout_signal_callback_add
      (obj, "drag,set", "efl.dragable.hbar",
      _efl_ui_view_list_edje_drag_cb, obj);
@@ -574,8 +565,8 @@ _efl_ui_view_list_edje_object_detach(Evas_Object *obj)
 
    //Horizontal bar
    efl_layout_signal_callback_del
-       (obj, "drag", "efl.dragable.hbar", _efl_ui_view_list_hbar_drag_cb,
-     obj);
+       (obj, "drag", "efl.dragable.hbar",
+       _efl_ui_view_list_hbar_drag_cb, obj);
    efl_layout_signal_callback_del
      (obj, "drag,set", "efl.dragable.hbar",
      _efl_ui_view_list_edje_drag_cb, obj);
@@ -632,14 +623,11 @@ _efl_ui_view_list_efl_canvas_group_group_add(Eo *obj, Efl_Ui_View_List_Data *pd)
    edje_object_thaw(wd->resize_obj);
    efl_gfx_stack_raise((Eo *)o);
 
-   pd->mode = ELM_LIST_COMPRESS;
-
    efl_gfx_entity_visible_set(pd->pan_obj, EINA_TRUE);
 
    edje_object_size_min_calc(wd->resize_obj, &min.w, &min.h);
    efl_gfx_size_hint_restricted_min_set(obj, min);
 
-   efl_event_callback_add(obj, EFL_UI_EVENT_SCROLL, _scroll_cb, obj);
    efl_event_callback_add(obj, EFL_UI_SCROLLBAR_EVENT_BAR_SIZE_CHANGED,
                          _efl_ui_view_list_bar_size_changed_cb, obj);
    efl_event_callback_add(obj, EFL_UI_SCROLLBAR_EVENT_BAR_POS_CHANGED,
@@ -706,7 +694,6 @@ _efl_ui_view_list_efl_object_constructor(Eo *obj, Efl_Ui_View_List_Data *pd)
    pd->style = eina_stringshare_add(elm_widget_style_get(obj));
 
    pd->factory = NULL;
-   pd->orient = EFL_ORIENT_DOWN;
    pd->min.w = 0;
    pd->min.h = 0;
 
@@ -721,7 +708,6 @@ _efl_ui_view_list_efl_object_destructor(Eo *obj, Efl_Ui_View_List_Data *pd)
    efl_unref(pd->model);
    eina_stringshare_del(pd->style);
 
-   efl_event_callback_del(obj, EFL_UI_EVENT_SCROLL, _scroll_cb, obj);
    _efl_ui_view_list_edje_object_detach(obj);
 
    ELM_SAFE_FREE(pd->pan_obj, evas_object_del);
index 7560870..94c5d89 100644 (file)
@@ -5,26 +5,19 @@
 # include "elementary_config.h"
 #endif
 
-#define ELM_INTERFACE_ATSPI_ACCESSIBLE_PROTECTED
-#define ELM_INTERFACE_ATSPI_SELECTION_PROTECTED
-#define ELM_INTERFACE_ATSPI_WIDGET_ACTION_PROTECTED
-
 #include <Elementary.h>
 #include "elm_priv.h"
 
 typedef struct _Efl_Ui_View_List_Data Efl_Ui_View_List_Data;
-
 int efl_ui_view_list_item_index_get(Efl_Ui_View_List_LayoutItem *item);
 
-typedef struct _Efl_Ui_View_List_Data Efl_Ui_View_List_Data;
-
 #include "efl_ui_view_list_segarray.h"
 
 struct _Efl_Ui_View_List_Data
 {
    Eo                           *obj;
    Eo                           *scrl_mgr;
-   Efl_Ui_View_List_Pan              *pan_obj;
+   Efl_Ui_View_List_Pan         *pan_obj;
    Efl_Model                    *model;
 
    Eina_Stringshare             *style;
@@ -41,21 +34,13 @@ struct _Efl_Ui_View_List_Data
 
    Efl_Ui_Focus_Manager         *manager;
    Efl_Ui_View_List_Relayout    *relayout;
-
-   Efl_Orient                   orient;
-
-   int segarray_first;
-   Efl_Ui_View_List_SegArray         *segarray;
+   Efl_Ui_View_List_SegArray    *segarray;
+   int                          segarray_first;
 
    Elm_Object_Select_Mode       select_mode;
-   Elm_List_Mode                mode;
-
-   Eina_Rect                    gmt;
    Eina_Size2D                  min;
 
    Eina_Bool                    homogeneous : 1;
-   Eina_Bool                    recalc : 1;
-   Eina_Bool                    on_hold : 1;
    Eina_Bool                    scrl_freeze : 1;
 };
 
@@ -65,17 +50,6 @@ struct _Efl_Ui_View_List_Pan_Data
 {
    Eo                     *wobj;
    Eina_Rect              gmt;
-   Evas_Coord             move_diff;
-
-   Ecore_Job              *resize_job;
-};
-
-typedef struct _Efl_Ui_View_List_Slice Efl_Ui_View_List_Slice;
-
-struct _Efl_Ui_View_List_Slice
-{
-   Efl_Ui_View_List_Data       *pd;
-   int                    newstart, slicestart, newslice;
 };
 
 #define EFL_UI_VIEW_LIST_DATA_GET(o, ptr) \