diff options
| author | Guilherme Lepsch <lepsch@expertisesolutions.com.br> | 2015-02-05 11:23:41 +0100 |
|---|---|---|
| committer | Cedric BAIL <cedric@osg.samsung.com> | 2015-02-05 11:23:44 +0100 |
| commit | 629b5e3d2bc268b5e48b03c91ccabf030d2b7f51 (patch) | |
| tree | 677b37e05185ba3da10435ee657cff4aef34bebd /src/lib/eldbus | |
| parent | dad7df18b8e7c3cc2413e4fceec2f61d99ed012e (diff) | |
| download | efl-629b5e3d2bc268b5e48b03c91ccabf030d2b7f51.tar.gz | |
eldbus: production code inside Eina Safety Check assert.
Summary: Eina Safety Checks assert macros don't do anything (no-op) if EINA_SAFETY_CHECKS is undefined.
@fix
Reviewers: felipealmeida, larryolj, cedric
Reviewed By: cedric
Subscribers: cedric
Differential Revision: https://phab.enlightenment.org/D1926
Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
Diffstat (limited to 'src/lib/eldbus')
| -rw-r--r-- | src/lib/eldbus/eldbus_message_from_eina_value.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/lib/eldbus/eldbus_message_from_eina_value.c b/src/lib/eldbus/eldbus_message_from_eina_value.c index c4ef74a576..05bb9f5e41 100644 --- a/src/lib/eldbus/eldbus_message_from_eina_value.c +++ b/src/lib/eldbus/eldbus_message_from_eina_value.c @@ -45,9 +45,8 @@ static Eina_Bool _array_append(const char *type, const Eina_Value *value_array, Eldbus_Message_Iter *iter) { Eldbus_Message_Iter *array; - - EINA_SAFETY_ON_FALSE_RETURN_VAL( - eldbus_message_iter_arguments_append(iter, type, &array), EINA_FALSE); + Eina_Bool ok = eldbus_message_iter_arguments_append(iter, type, &array); + EINA_SAFETY_ON_FALSE_RETURN_VAL(ok, EINA_FALSE); DBG("array of type %c", type[1]); switch (type[1]) { |