rimmed pushed a commit to branch master.

http://git.enlightenment.org/tools/eflete.git/commit/?id=8b03d81a33acda0e1da40f3bb04ad0ab898bec75

commit 8b03d81a33acda0e1da40f3bb04ad0ab898bec75
Author: Andrii Kroitor <an.kroi...@samsung.com>
Date:   Mon Sep 19 17:12:16 2016 +0300

    signals: move used eflete_widget signals to signals struct
---
 src/bin/common/signals.c                      | 33 ++++++++++++++++++++++++++
 src/bin/common/signals.h                      | 34 +++++++++++++++++++++++++++
 src/bin/ui/colorclass_manager.c               |  4 ++--
 src/bin/ui/image_manager.c                    |  4 ++--
 src/bin/ui/modal_window/modal_window.c        | 11 ++++-----
 src/bin/ui/property/property_color_control.c  |  4 ++--
 src/bin/ui/property/property_common.c         | 10 ++++----
 src/bin/ui/property/property_common_image.c   |  2 +-
 src/bin/ui/property/property_common_tween.c   |  4 ++--
 src/bin/ui/property/property_image_selector.c |  2 +-
 src/bin/ui/sound_manager.c                    |  4 ++--
 src/bin/ui/style_manager.c                    |  4 ++--
 src/bin/ui/workspace/container.c              | 17 ++++----------
 src/bin/ui/workspace/groupview.c              |  6 ++---
 src/bin/ui/workspace/highlight.c              |  6 ++---
 src/bin/ui/workspace/workspace.c              |  2 +-
 16 files changed, 101 insertions(+), 46 deletions(-)

diff --git a/src/bin/common/signals.c b/src/bin/common/signals.c
index d1f9242..4407337 100644
--- a/src/bin/common/signals.c
+++ b/src/bin/common/signals.c
@@ -81,6 +81,39 @@ const Signals signals = {
           },
      },
 
+     .eflete = {
+          .modal_window = {
+               .done                   = "[eflete]done",
+               .cancel                 = "[eflete]cancel",
+               .show_animation_finished= "[eflete]show,animation,finished",
+               .hide_animation_finished= "[eflete]hide,animation,finished",
+          },
+          .container = {
+               .changed                = "[eflete]changed",
+               .handler_br_moved       = "[eflete]handler,BR,moved",
+          },
+          .highlight = {
+               .drag_start             = "[eflete]hl,drag,start",
+               .drag_stop              = "[eflete]hl,drag,stop",
+               .changed                = "[eflete]changed",
+          },
+          .property = {
+               .color_control = {
+                    .changed           = "[eflete]color,changed",
+                    .dismissed         = "[eflete]color,dismissed",
+               },
+               .image_normal_control = {
+                    .changed           = "[eflete]image,normal,changed",
+               },
+               .image_tween_control = {
+                    .changed           = "[eflete]image,tween,changed",
+               },
+               .image_selector = {
+                    .changed           = "[eflete]image,selector,changed",
+               },
+          },
+     },
+
      .elm = {
           .spinner = {
                .changed                = "changed",
diff --git a/src/bin/common/signals.h b/src/bin/common/signals.h
index 4fa4d26..35492be 100644
--- a/src/bin/common/signals.h
+++ b/src/bin/common/signals.h
@@ -80,6 +80,40 @@ typedef struct {
          } zoom;
       } workspace;
    } shortcut;
+
+   const struct {
+      const struct {
+         const char *done;
+         const char *cancel;
+         const char *show_animation_finished;
+         const char *hide_animation_finished;
+      } modal_window;
+      const struct {
+         const char *changed;
+         const char *handler_br_moved;
+      } container;
+      const struct {
+         const char *drag_start;
+         const char *drag_stop;
+         const char *changed;
+      } highlight;
+      const struct {
+         const struct {
+            const char *changed;
+            const char *dismissed;
+         } color_control;
+         const struct {
+            const char *changed;
+         } image_normal_control;
+         const struct {
+            const char *changed;
+         } image_tween_control;
+         const struct {
+            const char *changed;
+         } image_selector;
+      } property;
+   } eflete;
+
    const struct {
       const struct {
          const char *changed;
diff --git a/src/bin/ui/colorclass_manager.c b/src/bin/ui/colorclass_manager.c
index 312300a..757e0f2 100644
--- a/src/bin/ui/colorclass_manager.c
+++ b/src/bin/ui/colorclass_manager.c
@@ -443,8 +443,8 @@ colorclass_manager_add(void)
    /* Creating main layout of window */
    mng.win = mw_add();
    mw_title_set(mng.win, _("Color class manager"));
-   evas_object_smart_callback_add(mng.win, "cancel", _mw_cancel_cb, NULL);
-   evas_object_smart_callback_add(mng.win, "done", _mw_done_cb, NULL);
+   evas_object_smart_callback_add(mng.win, signals.eflete.modal_window.cancel, 
_mw_cancel_cb, NULL);
+   evas_object_smart_callback_add(mng.win, signals.eflete.modal_window.done, 
_mw_done_cb, NULL);
 #if !HAVE_TIZEN
    ic = elm_icon_add(mng.win);
    elm_icon_standard_set(ic, "image2");
diff --git a/src/bin/ui/image_manager.c b/src/bin/ui/image_manager.c
index 599dd00..3b0d6fa 100644
--- a/src/bin/ui/image_manager.c
+++ b/src/bin/ui/image_manager.c
@@ -511,8 +511,8 @@ image_manager_add(void)
 
    mng.win = mw_add();
    mw_title_set(mng.win, _("Image manager"));
-   evas_object_smart_callback_add(mng.win, "cancel", _mw_cancel_cb, NULL);
-   evas_object_smart_callback_add(mng.win, "done", _mw_done_cb, NULL);
+   evas_object_smart_callback_add(mng.win, signals.eflete.modal_window.cancel, 
_mw_cancel_cb, NULL);
+   evas_object_smart_callback_add(mng.win, signals.eflete.modal_window.done, 
_mw_done_cb, NULL);
 #if !HAVE_TIZEN
    ic = elm_icon_add(mng.win);
    elm_icon_standard_set(ic, "image2");
diff --git a/src/bin/ui/modal_window/modal_window.c 
b/src/bin/ui/modal_window/modal_window.c
index d68fca2..5871243 100644
--- a/src/bin/ui/modal_window/modal_window.c
+++ b/src/bin/ui/modal_window/modal_window.c
@@ -22,9 +22,6 @@
 #include "main_window.h"
 #include "shortcuts.h"
 
-#define SIGNAL_DONE "done"
-#define SIGNAL_CANCEL "cancel"
-
 static void
 _response_cb(void *data,
              Evas_Object *obj __UNUSED__,
@@ -87,7 +84,7 @@ _anim_show_finish(void *data,
 
    assert(mw != NULL);
 
-   evas_object_smart_callback_call(mw, "show,anim,finished", "eflete");
+   evas_object_smart_callback_call(mw, 
signals.eflete.modal_window.show_animation_finished, "eflete");
 
    img = elm_layout_content_get(obj, "elm.swallow.content");
    evas_object_image_source_visible_set(img, true);
@@ -107,7 +104,7 @@ _anim_hide_finish(void *data,
 
    assert(mw != NULL);
 
-   evas_object_smart_callback_call(mw, "hide,anim,finished", "eflete");
+   evas_object_smart_callback_call(mw, 
signals.eflete.modal_window.hide_animation_finished, "eflete");
 
    img = elm_layout_content_get(obj, "elm.swallow.content");
    evas_object_image_source_visible_set(img, true);
@@ -157,7 +154,7 @@ _mw_close(void *data,
    assert(mw != NULL);
 
    _anim_hide(elm_object_parent_widget_get(mw), evas_object_evas_get(mw), mw, 
NULL);
-   evas_object_smart_callback_call(mw, SIGNAL_CANCEL, NULL);
+   evas_object_smart_callback_call(mw, signals.eflete.modal_window.cancel, 
NULL);
    ui_menu_items_list_disable_set(ap.menu, MENU_ITEMS_LIST_MAIN, false);
 }
 
@@ -171,7 +168,7 @@ _mw_done(void *data,
    assert(mw != NULL);
 
    _anim_hide(elm_object_parent_widget_get(mw), evas_object_evas_get(mw), mw, 
NULL);
-   evas_object_smart_callback_call(mw, SIGNAL_DONE, NULL);
+   evas_object_smart_callback_call(mw, signals.eflete.modal_window.done, NULL);
    ui_menu_items_list_disable_set(ap.menu, MENU_ITEMS_LIST_MAIN, false);
 }
 
diff --git a/src/bin/ui/property/property_color_control.c 
b/src/bin/ui/property/property_color_control.c
index 50b5f7c..1005fdd 100644
--- a/src/bin/ui/property/property_color_control.c
+++ b/src/bin/ui/property/property_color_control.c
@@ -39,7 +39,7 @@ _on_color_change(void *data,
    c = evas_object_data_get(control, COLOR_DATA);
    elm_colorselector_color_get(obj, &c->r, &c->g, &c->b, &c->a);
    property_color_control_color_set(control, c->r, c->g, c->b, c->a);
-   evas_object_smart_callback_call(control, "changed", NULL);
+   evas_object_smart_callback_call(control, 
signals.eflete.property.color_control.changed, NULL);
 }
 
 static Eina_Bool
@@ -52,7 +52,7 @@ _on_dismissed(void *data,
    assert(control != NULL);
 
    elm_object_scroll_freeze_pop(control);
-   evas_object_smart_callback_call(control, "dismissed", NULL);
+   evas_object_smart_callback_call(control, 
signals.eflete.property.color_control.dismissed, NULL);
 
    evas_object_hide(obj);
    return true;
diff --git a/src/bin/ui/property/property_common.c 
b/src/bin/ui/property/property_common.c
index 12c53eb..67ba71f 100644
--- a/src/bin/ui/property/property_common.c
+++ b/src/bin/ui/property/property_common.c
@@ -386,8 +386,8 @@ _control_create(Property_Attribute *pa, Property_Action 
*action, Evas_Object *pa
          break;
       case PROPERTY_CONTROL_COLOR:
          content = property_color_control_add(parent);
-         evas_object_smart_callback_add(content, "changed", _start_change_cb, 
pa);
-         evas_object_smart_callback_add(content, "dismissed", _stop_cb, pa);
+         evas_object_smart_callback_add(content, 
signals.eflete.property.color_control.changed, _start_change_cb, pa);
+         evas_object_smart_callback_add(content, 
signals.eflete.property.color_control.dismissed, _stop_cb, pa);
          break;
       case PROPERTY_CONTROL_COLORSEL:
          content = elm_colorselector_add(parent);
@@ -426,15 +426,15 @@ _control_create(Property_Attribute *pa, Property_Action 
*action, Evas_Object *pa
          break;
       case PROPERTY_CONTROL_IMAGE_NORMAL:
          content = property_image_normal_control_add(parent);
-         evas_object_smart_callback_add(content, "image,changed", 
_start_change_stop_cb, pa);
+         evas_object_smart_callback_add(content, 
signals.eflete.property.image_normal_control.changed, _start_change_stop_cb, 
pa);
          break;
       case PROPERTY_CONTROL_IMAGE_TWEEN:
          content = property_image_tween_control_add(parent);
-         evas_object_smart_callback_add(content, "image,tween,changed", 
_start_change_stop_cb, pa);
+         evas_object_smart_callback_add(content, 
signals.eflete.property.image_tween_control.changed, _start_change_stop_cb, pa);
          break;
       case PROPERTY_CONTROL_IMAGE_SELECTOR:
          content = property_image_selector_get(parent);
-         evas_object_smart_callback_add(content, "changed,demo,image", 
_start_change_stop_cb, pa);
+         evas_object_smart_callback_add(content, 
signals.eflete.property.image_selector.changed, _start_change_stop_cb, pa);
          evas_object_show(content);
          break;
       case PROPERTY_CONTROL_NONE:
diff --git a/src/bin/ui/property/property_common_image.c 
b/src/bin/ui/property/property_common_image.c
index a1ce325..96d644e 100644
--- a/src/bin/ui/property/property_common_image.c
+++ b/src/bin/ui/property/property_common_image.c
@@ -41,7 +41,7 @@ TODO("apply after helper popup would be fixed")
    if (strcmp(value, selected) == 0) return true;
    property_entry_set(data, selected);
 
-   evas_object_smart_callback_call(data, "image,changed", NULL);
+   evas_object_smart_callback_call(data, 
signals.eflete.property.image_normal_control.changed, NULL);
    elm_object_scroll_freeze_pop(data);
 
    return true;
diff --git a/src/bin/ui/property/property_common_tween.c 
b/src/bin/ui/property/property_common_tween.c
index b43a3af..b3d77de 100644
--- a/src/bin/ui/property/property_common_tween.c
+++ b/src/bin/ui/property/property_common_tween.c
@@ -90,7 +90,7 @@ _del_tween_image(void *data,
      }
 
    eina_list_free(selected);
-   evas_object_smart_callback_call(control, "image,tween,changed", NULL);
+   evas_object_smart_callback_call(control, 
signals.eflete.property.image_tween_control.changed, NULL);
 }
 
 static Eina_Bool
@@ -110,7 +110,7 @@ _on_image_editor_tween_done(void *data,
         added_tweens = eina_list_append(added_tweens, name);
      }
 
-   evas_object_smart_callback_call(control, "image,tween,changed", NULL);
+   evas_object_smart_callback_call(control, 
signals.eflete.property.image_tween_control.changed, NULL);
 TODO("apply when popup will be fixed");
 //   elm_object_scroll_freeze_pop(tween_list);
 
diff --git a/src/bin/ui/property/property_image_selector.c 
b/src/bin/ui/property/property_image_selector.c
index 1811cab..9369ade 100644
--- a/src/bin/ui/property/property_image_selector.c
+++ b/src/bin/ui/property/property_image_selector.c
@@ -38,7 +38,7 @@ _on_image_done(void *data,
    if (strcmp(value, selected) == 0) return true;
    property_entry_set(data, selected);
 
-   evas_object_smart_callback_call(data, "changed,demo,image", NULL);
+   evas_object_smart_callback_call(data, 
signals.eflete.property.image_selector.changed, NULL);
 
    return true;
 }
diff --git a/src/bin/ui/sound_manager.c b/src/bin/ui/sound_manager.c
index b32cf85..3bf1fc8 100644
--- a/src/bin/ui/sound_manager.c
+++ b/src/bin/ui/sound_manager.c
@@ -556,8 +556,8 @@ sound_manager_add(void)
 
    mng.win = mw_add();
    mw_title_set(mng.win, _("Sound manager"));
-   evas_object_smart_callback_add(mng.win, "cancel", _mw_cancel_cb, NULL);
-   evas_object_smart_callback_add(mng.win, "done", _mw_done_cb, NULL);
+   evas_object_smart_callback_add(mng.win, signals.eflete.modal_window.cancel, 
_mw_cancel_cb, NULL);
+   evas_object_smart_callback_add(mng.win, signals.eflete.modal_window.done, 
_mw_done_cb, NULL);
 #if !HAVE_TIZEN
    ic = elm_icon_add(mng.win);
    elm_icon_standard_set(ic, "image2");
diff --git a/src/bin/ui/style_manager.c b/src/bin/ui/style_manager.c
index a9eddfb..b4099fa 100644
--- a/src/bin/ui/style_manager.c
+++ b/src/bin/ui/style_manager.c
@@ -811,8 +811,8 @@ style_manager_add()
 
    mng.win = mw_add();
    mw_title_set(mng.win, _("Textblock style manager"));
-   evas_object_smart_callback_add(mng.win, "cancel", _mw_cancel_cb, NULL);
-   evas_object_smart_callback_add(mng.win, "done", _mw_done_cb, NULL);
+   evas_object_smart_callback_add(mng.win, signals.eflete.modal_window.cancel, 
_mw_cancel_cb, NULL);
+   evas_object_smart_callback_add(mng.win, signals.eflete.modal_window.done, 
_mw_done_cb, NULL);
 #if !HAVE_TIZEN
    ic = elm_icon_add(mng.win);
    elm_icon_standard_set(ic, "image2");
diff --git a/src/bin/ui/workspace/container.c b/src/bin/ui/workspace/container.c
index a071903..fd46745 100644
--- a/src/bin/ui/workspace/container.c
+++ b/src/bin/ui/workspace/container.c
@@ -26,18 +26,9 @@
 
 typedef struct _Container_Smart_Data Container_Smart_Data;
 
-static const char SIG_CHANGED[] = "container,changed";
-static const char SIG_HANDLER_BR_MOVE[] = "handler,BR,moved";
-
 static const char TEXT_TOOLTIP[] = "eflete.size.tooltip";
 static const char SWALLOW[] = "eflete.swallow.container";
 
-static const Evas_Smart_Cb_Description _smart_callbacks[] = {
-   {SIG_CHANGED, "(iiii)"},
-   {SIG_HANDLER_BR_MOVE, "(ii)"},
-   {NULL, NULL}
-};
-
 struct _Container_Smart_Data
 {
    Evas_Object_Smart_Clipped_Data base;
@@ -81,7 +72,7 @@ struct _Container_Smart_Data
  * object that is inherited from general Smart Object. */
 EVAS_SMART_SUBCLASS_NEW(MY_CLASS_NAME, _container,
                         Evas_Smart_Class, Evas_Smart_Class,
-                        evas_object_smart_clipped_class_get, _smart_callbacks);
+                        evas_object_smart_clipped_class_get, NULL);
 
 static void
 _mouse_down_hRB_cb(void *data,
@@ -127,7 +118,7 @@ _mouse_move_hBR_cb(void *data,
    sd->downx = ev->cur.canvas.x;
    sd->downy = ev->cur.canvas.y;
 
-   evas_object_smart_callback_call(o, SIG_HANDLER_BR_MOVE, &sd->zoom1);
+   evas_object_smart_callback_call(o, 
signals.eflete.container.handler_br_moved, &sd->zoom1);
    evas_object_smart_changed(o);
 }
 
@@ -258,7 +249,7 @@ _container_smart_move(Evas_Object *o,
    evas_object_move(sd->container, sd->size.x, sd->size.y);
    evas_object_move(sd->handler_BR.obj, sd->size.x + sd->size.w, sd->size.y + 
sd->size.h);
 
-   evas_object_smart_callback_call(o, SIG_CHANGED, &sd->size);
+   evas_object_smart_callback_call(o, signals.eflete.container.changed, 
&sd->size);
 }
 
 static void
@@ -363,7 +354,7 @@ _container_smart_calculate(Evas_Object *o)
    snprintf(buff, 16, "%i %i", sd->zoom1.w, sd->zoom1.h);
    edje_object_part_text_set(sd->container, TEXT_TOOLTIP, buff);
 
-   evas_object_smart_callback_call(o, SIG_CHANGED, &sd->zoom1);
+   evas_object_smart_callback_call(o, signals.eflete.container.changed, 
&sd->zoom1);
 
    if ((int)sd->zoom > 1) divisior = (int)sd->zoom;
    sd->dx = sd->dx % divisior;
diff --git a/src/bin/ui/workspace/groupview.c b/src/bin/ui/workspace/groupview.c
index aa2db0d..314e6ce 100644
--- a/src/bin/ui/workspace/groupview.c
+++ b/src/bin/ui/workspace/groupview.c
@@ -291,9 +291,9 @@ groupview_add(Evas_Object *parent, Group *group)
 
    sd->highlight = highlight_add(sd->box);
    evas_object_color_set(sd->highlight, HIGHLIGHT_COLOR);
-   evas_object_smart_callback_add(sd->highlight, "hl,drag,start", 
_hl_part_drag_start_cb, sd);
-   evas_object_smart_callback_add(sd->highlight, "hl,changed", 
_hl_part_changed_cb, sd);
-   evas_object_smart_callback_add(sd->highlight, "hl,drag,stop", 
_hl_part_drag_stop_cb, sd);
+   evas_object_smart_callback_add(sd->highlight, 
signals.eflete.highlight.drag_start, _hl_part_drag_start_cb, sd);
+   evas_object_smart_callback_add(sd->highlight, 
signals.eflete.highlight.changed, _hl_part_changed_cb, sd);
+   evas_object_smart_callback_add(sd->highlight, 
signals.eflete.highlight.drag_stop, _hl_part_drag_stop_cb, sd);
    evas_object_smart_member_add(sd->highlight, obj);
 
    /* hide the editing object by using clipper (clipper is small, it's size is 
0,0)
diff --git a/src/bin/ui/workspace/highlight.c b/src/bin/ui/workspace/highlight.c
index 7824ae8..0f6c47a 100644
--- a/src/bin/ui/workspace/highlight.c
+++ b/src/bin/ui/workspace/highlight.c
@@ -236,7 +236,7 @@ _handler_down_cb(void *data,
 
    highlight->clicked = true;
    evas_object_smart_callback_call(highlight->smart_object,
-                                   "hl,drag,start", highlight->events);
+                                   signals.eflete.highlight.drag_start, 
highlight->events);
 }
 
 static void
@@ -435,7 +435,7 @@ _handler_up_cb(void *data,
    handler->highlight->clicked = false;
 
    evas_object_smart_callback_call(handler->highlight->smart_object,
-                                   "hl,drag,stop", handler->highlight->events);
+                                   signals.eflete.highlight.drag_stop, 
handler->highlight->events);
 }
 
 /*
@@ -573,7 +573,7 @@ _smart_calc(Evas_Object *obj)
 
    if (highlight->clicked)
      evas_object_smart_callback_call(highlight->smart_object,
-                                     "hl,changed", highlight->events);
+                                     signals.eflete.highlight.changed, 
highlight->events);
 }
 
 /* create and setup a new example smart object's internals */
diff --git a/src/bin/ui/workspace/workspace.c b/src/bin/ui/workspace/workspace.c
index 11d9fda..a644ce4 100644
--- a/src/bin/ui/workspace/workspace.c
+++ b/src/bin/ui/workspace/workspace.c
@@ -1100,7 +1100,7 @@ _scroll_area_add(Workspace_Data *wd, Scroll_Area *area, 
Eina_Bool scale_rel)
    area->container = container_add(area->scroller);
    evas_object_size_hint_weight_set(area->container, EVAS_HINT_EXPAND, 
EVAS_HINT_EXPAND);
    container_handler_size_set(area->container, 8, 8);
-   evas_object_smart_callback_add(area->container, "container,changed", 
_container_changed, wd);
+   evas_object_smart_callback_add(area->container, 
signals.eflete.container.changed, _container_changed, wd);
    elm_object_content_set(area->helper, area->container);
    container_container_size_set(area->container, 350, 350);
 

-- 


Reply via email to