From 629b5e3d2bc268b5e48b03c91ccabf030d2b7f51 Mon Sep 17 00:00:00 2001 From: Guilherme Lepsch Date: Thu, 5 Feb 2015 11:23:41 +0100 Subject: [PATCH] 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 --- 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 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]) {