summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTom Hacohen <tom@stosb.com>2015-04-17 14:31:19 +0100
committerTom Hacohen <tom@stosb.com>2015-04-17 14:31:21 +0100
commit9570461440db497be65ee1ab4b5d9974e87f9a7d (patch)
tree3b9d0037e51c85b462624a890b09b9472c0ac24a
parent963fe509ca1acc5bef1585bb65e8264a5036697a (diff)
Eo: Add eo_do_part.devs/tasn/part_get
This is a convenience macro to be used by the common pattern of getting a part and then immediately calling functions on it. For example, without this macro, you'd have to write code like: Eo *part; eo_do(obj, part = efl_part_name_get("partname")); eo_do(part, a_set(7)); while using the helper function trims it to: eo_do_part(obj, efl_part_name_get("partname"), a_set(7)); @feature
-rw-r--r--src/lib/eo/Eo.h7
-rw-r--r--src/tests/eo/suite/eo_test_class_simple.c10
-rw-r--r--src/tests/eo/suite/eo_test_class_simple.h1
-rw-r--r--src/tests/eo/suite/eo_test_general.c34
4 files changed, 52 insertions, 0 deletions
diff --git a/src/lib/eo/Eo.h b/src/lib/eo/Eo.h
index 46e9d73ef8..59a8528e7f 100644
--- a/src/lib/eo/Eo.h
+++ b/src/lib/eo/Eo.h
@@ -607,6 +607,13 @@ EAPI Eo * _eo_add_end(void);
607 607
608#define eo_do_super_ret(eoid, clsid, ret_tmp, func) _eo_do_common_ret(eoid, clsid, EINA_TRUE, ret_tmp, func) 608#define eo_do_super_ret(eoid, clsid, ret_tmp, func) _eo_do_common_ret(eoid, clsid, EINA_TRUE, ret_tmp, func)
609 609
610#define eo_do_part(eoid, part_func, ...) \
611 do { \
612 Eo *__eo_part = eoid; \
613 eo_do(eoid, __eo_part = part_func); \
614 eo_do(__eo_part, __VA_ARGS__); \
615 } while (0)
616
610/*****************************************************************************/ 617/*****************************************************************************/
611 618
612/** 619/**
diff --git a/src/tests/eo/suite/eo_test_class_simple.c b/src/tests/eo/suite/eo_test_class_simple.c
index cbefee7d1d..4bc3904acb 100644
--- a/src/tests/eo/suite/eo_test_class_simple.c
+++ b/src/tests/eo/suite/eo_test_class_simple.c
@@ -45,6 +45,15 @@ _class_hi_print(Eo_Class *klass, void *data EINA_UNUSED)
45 return EINA_TRUE; 45 return EINA_TRUE;
46} 46}
47 47
48EO_FUNC_BODYV(simple_part_get, Eo *, NULL, EO_FUNC_CALL(name), const char *name);
49
50static Eo *
51_part_get(Eo *obj, void *class_data EINA_UNUSED, const char *name EINA_UNUSED)
52{
53 /* A normal part get will do something saner, we just create objects. */
54 return eo_add(SIMPLE_CLASS, obj);
55}
56
48EO_VOID_FUNC_BODYV(simple_recursive, EO_FUNC_CALL(n), int n); 57EO_VOID_FUNC_BODYV(simple_recursive, EO_FUNC_CALL(n), int n);
49 58
50static void 59static void
@@ -83,6 +92,7 @@ static Eo_Op_Description op_descs[] = {
83 EO_OP_FUNC(simple_a_print, _a_print, "Print property a"), 92 EO_OP_FUNC(simple_a_print, _a_print, "Print property a"),
84 EO_OP_CLASS_FUNC(simple_class_hi_print, _class_hi_print, "Print property a"), 93 EO_OP_CLASS_FUNC(simple_class_hi_print, _class_hi_print, "Print property a"),
85 EO_OP_FUNC(simple_recursive, _recursive, "Recursive function"), 94 EO_OP_FUNC(simple_recursive, _recursive, "Recursive function"),
95 EO_OP_FUNC(simple_part_get, _part_get, "Part getter"),
86 EO_OP_FUNC(simple_pure_virtual, NULL, "Pure Virtual function"), 96 EO_OP_FUNC(simple_pure_virtual, NULL, "Pure Virtual function"),
87 EO_OP_SENTINEL 97 EO_OP_SENTINEL
88}; 98};
diff --git a/src/tests/eo/suite/eo_test_class_simple.h b/src/tests/eo/suite/eo_test_class_simple.h
index 7d774b761f..2fce591372 100644
--- a/src/tests/eo/suite/eo_test_class_simple.h
+++ b/src/tests/eo/suite/eo_test_class_simple.h
@@ -13,6 +13,7 @@ EAPI Eina_Bool simple_class_hi_print(void);
13EAPI void simple_recursive(int n); 13EAPI void simple_recursive(int n);
14EAPI void simple_pure_virtual(void); 14EAPI void simple_pure_virtual(void);
15EAPI void simple_no_implementation(void); 15EAPI void simple_no_implementation(void);
16EAPI Eo *simple_part_get(const char *name);
16 17
17extern const Eo_Event_Description _EV_A_CHANGED; 18extern const Eo_Event_Description _EV_A_CHANGED;
18#define EV_A_CHANGED (&(_EV_A_CHANGED)) 19#define EV_A_CHANGED (&(_EV_A_CHANGED))
diff --git a/src/tests/eo/suite/eo_test_general.c b/src/tests/eo/suite/eo_test_general.c
index baef2fe569..91ecd1c4c8 100644
--- a/src/tests/eo/suite/eo_test_general.c
+++ b/src/tests/eo/suite/eo_test_general.c
@@ -937,6 +937,39 @@ START_TEST(eo_add_failures)
937} 937}
938END_TEST 938END_TEST
939 939
940START_TEST(eo_parts)
941{
942 int a = 0;
943
944 eo_init();
945
946 Eo *obj = eo_add(SIMPLE_CLASS, NULL);
947
948 eo_do(obj, simple_a_set(3), a = simple_a_get());
949 ck_assert_int_eq(a, 3);
950
951 eo_do_part(obj, simple_part_get("test"),
952 simple_a_set(7),
953 a = simple_a_get()
954 );
955 ck_assert_int_eq(a, 7);
956
957 eo_do(obj, simple_a_set(3), a = simple_a_get());
958 ck_assert_int_eq(a, 3);
959
960 /* Faking a call, just asserting NULL as the part to check default values. */
961 eo_do_part(obj, NULL,
962 simple_a_set(7),
963 a = simple_a_get()
964 );
965 ck_assert_int_eq(a, 0);
966
967 eo_del(obj);
968
969 eo_shutdown();
970}
971END_TEST
972
940void eo_test_general(TCase *tc) 973void eo_test_general(TCase *tc)
941{ 974{
942 tcase_add_test(tc, eo_simple); 975 tcase_add_test(tc, eo_simple);
@@ -954,4 +987,5 @@ void eo_test_general(TCase *tc)
954 tcase_add_test(tc, eo_add_do_and_custom); 987 tcase_add_test(tc, eo_add_do_and_custom);
955 tcase_add_test(tc, eo_pointers_indirection); 988 tcase_add_test(tc, eo_pointers_indirection);
956 tcase_add_test(tc, eo_add_failures); 989 tcase_add_test(tc, eo_add_failures);
990 tcase_add_test(tc, eo_parts);
957} 991}