From: Oleg Zhurakivskyy Date: Tue, 10 Apr 2012 12:17:26 +0000 (+0300) Subject: call-forwarding: Streamline cf_find_timeout() logic X-Git-Tag: 1.7~97 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d797a868c058e03c016f4eb517ead0f47abbf44c;p=platform%2Fupstream%2Fofono.git call-forwarding: Streamline cf_find_timeout() logic --- diff --git a/src/call-forwarding.c b/src/call-forwarding.c index 0733f76..2a7bff0 100644 --- a/src/call-forwarding.c +++ b/src/call-forwarding.c @@ -108,16 +108,11 @@ static struct ofono_call_forwarding_condition *cf_cond_find(GSList *l, int cls) return NULL; } -static int cf_find_timeout(GSList *cf_list, int cls) +static int cf_cond_find_timeout(GSList *l, int cls) { - struct ofono_call_forwarding_condition *c; - - c = cf_cond_find(cf_list, cls); - - if (c == NULL) - return DEFAULT_NO_REPLY_TIMEOUT; + struct ofono_call_forwarding_condition *cond = cf_cond_find(l, cls); - return c->time; + return cond ? cond->time : DEFAULT_NO_REPLY_TIMEOUT; } static void cf_cond_list_print(GSList *l) @@ -820,7 +815,7 @@ static DBusMessage *cf_set_property(DBusConnection *conn, DBusMessage *msg, if (number[0] != '\0') string_to_phone_number(number, &ph); - timeout = cf_find_timeout(cf->cf_conditions[type], cls); + timeout = cf_cond_find_timeout(cf->cf_conditions[type], cls); return set_property_request(cf, msg, type, cls, &ph, timeout);