jackdanielz pushed a commit to branch master. http://git.enlightenment.org/core/efl.git/commit/?id=d833722e6ee2fb2848ab28e940aac6dd2a4b2969
commit d833722e6ee2fb2848ab28e940aac6dd2a4b2969 Author: Avi Levin <avi.le...@samsung.com> Date: Thu Jun 11 15:20:13 2015 +0300 emotion: revert changes to emotion test as discussed on ML and chat We want a separate test for Eo callbacks, as we still want to verify legacy callbacks. --- src/tests/emotion/emotion_test_main.c | 110 +++++++++++----------------------- 1 file changed, 36 insertions(+), 74 deletions(-) diff --git a/src/tests/emotion/emotion_test_main.c b/src/tests/emotion/emotion_test_main.c index 3d7d4a9..24ab7db 100644 --- a/src/tests/emotion/emotion_test_main.c +++ b/src/tests/emotion/emotion_test_main.c @@ -1,11 +1,7 @@ #ifdef HAVE_CONFIG_H -#include "config.h" -#else -#define EFL_EO_API_SUPPORT -#define EFL_BETA_API_SUPPORT +# include "config.h" #endif -#include <Eo.h> #include <stdlib.h> #include <stdio.h> #include <string.h> @@ -329,9 +325,8 @@ video_obj_time_changed(Evas_Object *obj, Evas_Object *edje) edje_object_part_text_set(edje, "video_progress_txt", buf); } -static Eina_Bool -video_obj_frame_decode_cb(void *data, - Eo *obj, const Eo_Event_Description *desc EINA_UNUSED, void *event_info EINA_UNUSED) +static void +video_obj_frame_decode_cb(void *data, Evas_Object *obj, void *event_info EINA_UNUSED) { video_obj_time_changed(obj, data); @@ -343,12 +338,10 @@ video_obj_frame_decode_cb(void *data, printf("FPS: %3.3f\n", 1.0 / (t - pt)); pt = t; } - return EINA_TRUE; } -static Eina_Bool -video_obj_frame_resize_cb(void *data, - Eo *obj, const Eo_Event_Description *desc EINA_UNUSED, void *event_info EINA_UNUSED) +static void +video_obj_frame_resize_cb(void *data, Evas_Object *obj, void *event_info EINA_UNUSED) { Evas_Object *oe; int iw, ih; @@ -366,33 +359,24 @@ video_obj_frame_resize_cb(void *data, evas_object_resize(oe, w, h); evas_object_size_hint_min_set(obj, 0, 0); edje_object_part_swallow(oe, "video_swallow", obj); - - return EINA_TRUE; } -static Eina_Bool -video_obj_length_change_cb(void *data, - Eo *obj, const Eo_Event_Description *desc EINA_UNUSED, void *event_info EINA_UNUSED) +static void +video_obj_length_change_cb(void *data, Evas_Object *obj, void *event_info EINA_UNUSED) { printf("len change!\n"); video_obj_time_changed(obj, data); - - return EINA_TRUE; } -static Eina_Bool -video_obj_position_update_cb(void *data, - Eo *obj, const Eo_Event_Description *desc EINA_UNUSED, void *event_info EINA_UNUSED) +static void +video_obj_position_update_cb(void *data, Evas_Object *obj, void *event_info EINA_UNUSED) { printf("pos up!\n"); video_obj_time_changed(obj, data); - - return EINA_TRUE; } -static Eina_Bool -video_obj_stopped_cb(void *data EINA_UNUSED, - Eo *obj, const Eo_Event_Description *desc EINA_UNUSED, void *event_info EINA_UNUSED) +static void +video_obj_stopped_cb(void *data EINA_UNUSED, Evas_Object *obj, void *event_info EINA_UNUSED) { printf("video stopped!!!\n"); if (loop) @@ -400,73 +384,54 @@ video_obj_stopped_cb(void *data EINA_UNUSED, emotion_object_position_set(obj, 0.0); emotion_object_play_set(obj, 1); } - - return EINA_TRUE; } -static Eina_Bool -video_obj_channels_cb(void *data EINA_UNUSED, - Eo *obj, const Eo_Event_Description *desc EINA_UNUSED, void *event_info EINA_UNUSED) +static void +video_obj_channels_cb(void *data EINA_UNUSED, Evas_Object *obj, void *event_info EINA_UNUSED) { printf("channels changed: [AUD %i][VID %i][SPU %i]\n", emotion_object_audio_channel_count(obj), emotion_object_video_channel_count(obj), emotion_object_spu_channel_count(obj)); - - return EINA_TRUE; } -static Eina_Bool - video_obj_title_cb(void *data EINA_UNUSED, - Eo *obj, const Eo_Event_Description *desc EINA_UNUSED, void *event_info EINA_UNUSED) +static void +video_obj_title_cb(void *data EINA_UNUSED, Evas_Object *obj, void *event_info EINA_UNUSED) { printf("video title to: \"%s\"\n", emotion_object_title_get(obj)); - - return EINA_TRUE; } -static Eina_Bool -video_obj_progress_cb(void *data EINA_UNUSED, - Eo *obj, const Eo_Event_Description *desc EINA_UNUSED, void *event_info EINA_UNUSED) +static void +video_obj_progress_cb(void *data EINA_UNUSED, Evas_Object *obj, void *event_info EINA_UNUSED) { printf("progress: \"%s\" %3.3f\n", emotion_object_progress_info_get(obj), emotion_object_progress_status_get(obj)); - - return EINA_TRUE; } -static Eina_Bool -video_obj_ref_cb(void *data EINA_UNUSED, - Eo *obj, const Eo_Event_Description *desc EINA_UNUSED, void *event_info EINA_UNUSED) +static void +video_obj_ref_cb(void *data EINA_UNUSED, Evas_Object *obj, void *event_info EINA_UNUSED) { printf("video ref to: \"%s\" %i\n", emotion_object_ref_file_get(obj), emotion_object_ref_num_get(obj)); - - return EINA_TRUE; } -static Eina_Bool -video_obj_button_num_cb(void *data EINA_UNUSED, - Eo *obj, const Eo_Event_Description *desc EINA_UNUSED, void *event_info EINA_UNUSED) +static void +video_obj_button_num_cb(void *data EINA_UNUSED, Evas_Object *obj, void *event_info EINA_UNUSED) { printf("video spu buttons to: %i\n", emotion_object_spu_button_count_get(obj)); - - return EINA_TRUE; } -static Eina_Bool -video_obj_button_cb(void *data EINA_UNUSED, - Eo *obj, const Eo_Event_Description *desc EINA_UNUSED, void *event_info EINA_UNUSED) +static void +video_obj_button_cb(void *data EINA_UNUSED, Evas_Object *obj, void *event_info EINA_UNUSED) { printf("video selected spu button: %i\n", emotion_object_spu_button_get(obj)); - - return EINA_TRUE; } + static void video_obj_signal_play_cb(void *data, Evas_Object *o, const char *emission EINA_UNUSED, const char *source EINA_UNUSED) { @@ -611,20 +576,6 @@ video_obj_signal_frame_move_cb(void *data EINA_UNUSED, Evas_Object *o, const cha } } -static const Eo_Callback_Array_Item emotion_object_test_callbacks[] = { - { EMOTION_OBJECT_EVENT_FRAME_DECODE, video_obj_frame_decode_cb }, - { EMOTION_OBJECT_EVENT_FRAME_RESIZE, video_obj_frame_resize_cb }, - { EMOTION_OBJECT_EVENT_LENGTH_CHANGE, video_obj_length_change_cb }, - { EMOTION_OBJECT_EVENT_POSITION_UPDATE, video_obj_position_update_cb }, - { EMOTION_OBJECT_EVENT_DECODE_STOP, video_obj_stopped_cb }, - { EMOTION_OBJECT_EVENT_CHANNELS_CHANGE, video_obj_channels_cb }, - { EMOTION_OBJECT_EVENT_TITLE_CHANGE, video_obj_title_cb }, - { EMOTION_OBJECT_EVENT_PROGRESS_CHANGE, video_obj_progress_cb }, - { EMOTION_OBJECT_EVENT_REF_CHANGE, video_obj_ref_cb }, - { EMOTION_OBJECT_EVENT_BUTTON_NUM_CHANGE, video_obj_button_num_cb }, - { EMOTION_OBJECT_EVENT_BUTTON_CHANGE, video_obj_button_cb }, - { NULL, NULL } -}; static void init_video_object(const char *module_filename, const char *filename) @@ -670,7 +621,18 @@ init_video_object(const char *module_filename, const char *filename) edje_object_size_min_calc(oe, &w, &h); evas_object_resize(oe, w, h); - eo_do(o, eo_event_callback_array_add(emotion_object_test_callbacks, oe)); + evas_object_smart_callback_add(o, "frame_decode", video_obj_frame_decode_cb, oe); + evas_object_smart_callback_add(o, "frame_resize", video_obj_frame_resize_cb, oe); + evas_object_smart_callback_add(o, "length_change", video_obj_length_change_cb, oe); + evas_object_smart_callback_add(o, "position_update", video_obj_position_update_cb, oe); + + evas_object_smart_callback_add(o, "decode_stop", video_obj_stopped_cb, oe); + evas_object_smart_callback_add(o, "channels_change", video_obj_channels_cb, oe); + evas_object_smart_callback_add(o, "title_change", video_obj_title_cb, oe); + evas_object_smart_callback_add(o, "progress_change", video_obj_progress_cb, oe); + evas_object_smart_callback_add(o, "ref_change", video_obj_ref_cb, oe); + evas_object_smart_callback_add(o, "button_num_change", video_obj_button_num_cb, oe); + evas_object_smart_callback_add(o, "button_change", video_obj_button_cb, oe); edje_object_signal_callback_add(oe, "video_control", "play", video_obj_signal_play_cb, o); edje_object_signal_callback_add(oe, "video_control", "pause", video_obj_signal_pause_cb, o); --