diff options
| author | Gustavo Sverzut Barbieri <barbieri@profusion.mobi> | 2016-08-18 01:01:25 -0300 |
|---|---|---|
| committer | Gustavo Sverzut Barbieri <barbieri@profusion.mobi> | 2016-08-18 02:13:32 -0300 |
| commit | c88b027dc1ac0ffa17022ebebdad1f10687622a3 (patch) | |
| tree | bfd7b2c7e8de02986d05453efdc087cd6461de7c | |
| parent | e3e0a6bd65e7168bf2ca53b305708c52d9951b1b (diff) | |
| download | efl-c88b027dc1ac0ffa17022ebebdad1f10687622a3.tar.gz | |
efl_loop_user: error if parent is not a loop provider.
it was silently ignoring the parent, which led me to hours trying to
investigate why my code wasn't working just to realize my parent
wasn't being assigned and refcounts were screwed.
| -rw-r--r-- | src/lib/ecore/efl_loop_user.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/lib/ecore/efl_loop_user.c b/src/lib/ecore/efl_loop_user.c index 78ac3a83d7..530772fc64 100644 --- a/src/lib/ecore/efl_loop_user.c +++ b/src/lib/ecore/efl_loop_user.c @@ -21,7 +21,10 @@ static void _efl_loop_user_efl_object_parent_set(Eo *obj, Efl_Loop_User_Data *pd EINA_UNUSED, Efl_Object *parent) { if (parent != NULL && efl_provider_find(parent, EFL_LOOP_CLASS) == NULL) - return ; + { + ERR("parent=%p is not a provider of EFL_LOOP_CLASS!", parent); + return; + } efl_parent_set(efl_super(obj, EFL_LOOP_USER_CLASS), parent); } |