cedric pushed a commit to branch master. http://git.enlightenment.org/core/efl.git/commit/?id=629b5e3d2bc268b5e48b03c91ccabf030d2b7f51
commit 629b5e3d2bc268b5e48b03c91ccabf030d2b7f51 Author: Guilherme Lepsch <[email protected]> Date: Thu Feb 5 11:23:41 2015 +0100 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 <[email protected]> --- src/lib/eldbus/eldbus_message_from_eina_value.c | 5 ++--- 1 file 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 c4ef74a..05bb9f5 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]) { --
