enlightenment/efl
enlightenment
/
efl
8
7
Fork 14
Code Issues 13 Pull Requests 3 Projects Releases Wiki Activity

eo_test_general.c: Make eo_signals tests pass on Windows 

Summary:
EFL_CALLBACKS_ARRAY_DEFINE reorders the callbacks according to
efl_callbacks_cmp. efl_callbacks_cmp compares the address of the desc
field, which depends on the memory layout generated by the linker.

To make the test run deterministically, we define the array of callbacks
manually.

Reviewers: vtorri, felipealmeida, raster

Reviewed By: raster

Subscribers: cedric, #reviewers, #committers, jptiz, felipealmeida

Tags: #efl

Differential Revision: https://phab.enlightenment.org/D12043
This commit is contained in:
Wander Lairson Costa 2020-11-04 18:25:37 +00:00 committed by Carsten Haitzler (Rasterman)
parent 888e1e7401
commit fc949660f7
1 changed files with 20 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
src/tests/eo/suite/eo_test_general.c
View File

@ -199,11 +199,26 @@ _eo_signals_cb_added_deled(void *data EINA_UNUSED, const Efl_Event *event)
fail_if(callback_array->func != _eo_signals_cb_added_deled);
}
EFL_CALLBACKS_ARRAY_DEFINE(_eo_signals_callbacks,
{ EV_A_CHANGED, _eo_signals_a_changed_cb },
{ EV_A_CHANGED, _eo_signals_a_changed_cb2 },
{ EV_A_CHANGED, _eo_signals_a_changed_never },
{ EFL_EVENT_DEL, _eo_signals_efl_del_cb });
// We don't use the EFL_CALLBACKS_ARRAY_DEFINE macro because
// we need the callbacks be called in a specific order
static Efl_Callback_Array_Item *
_eo_signals_callbacks(void)
{
static Efl_Callback_Array_Item items[] =
{
{ EV_A_CHANGED, _eo_signals_a_changed_cb },
{ EV_A_CHANGED, _eo_signals_a_changed_cb2 },
{ EV_A_CHANGED, _eo_signals_a_changed_never },
{ 0, _eo_signals_efl_del_cb },
{ 0, 0 },
};
// On Windows, because _EFL_EVENT_DEL is a symbol exported
// from the DLL, we can't assign from a context expression
items[3].desc = EFL_EVENT_DEL;
return items;
}
EFL_START_TEST(eo_signals)
{