diff options
author | Denis Kenzior <denkenz@gmail.com> | 2010-05-28 10:57:37 -0500 |
---|---|---|
committer | Denis Kenzior <denkenz@gmail.com> | 2010-05-28 11:29:35 -0500 |
commit | 5588433ad41e6e41c34eae3f0cba11c97d71180a (patch) | |
tree | 2cb35bff7ad8cfd6e7885ea6608024fa28917201 /src/stkutil.c | |
parent | a597318b78eda6e058b56a83f6bdf66ce1b0a232 (diff) | |
download | ofono-5588433ad41e6e41c34eae3f0cba11c97d71180a.tar.bz2 |
stkutil: Fix style in build_dataobj_location_info
Diffstat (limited to 'src/stkutil.c')
-rw-r--r-- | src/stkutil.c | 39 |
1 files changed, 27 insertions, 12 deletions
diff --git a/src/stkutil.c b/src/stkutil.c index d93f6e84..d4aa19a0 100644 --- a/src/stkutil.c +++ b/src/stkutil.c @@ -3413,18 +3413,33 @@ static gboolean build_dataobj_location_info(struct stk_tlv_builder *tlv, sim_encode_mcc_mnc(mccmnc, li->mcc, li->mnc); - return stk_tlv_builder_open_container(tlv, cr, tag, FALSE) && - stk_tlv_builder_append_bytes(tlv, mccmnc, 3) && - stk_tlv_builder_append_short(tlv, li->lac_tac) && - (li->has_ci == FALSE || - stk_tlv_builder_append_short(tlv, li->ci)) && - (li->has_ext_ci == FALSE || - stk_tlv_builder_append_short(tlv, li->ext_ci)) && - (li->has_eutran_ci == FALSE || - (stk_tlv_builder_append_short(tlv, li->eutran_ci >> 12) && - stk_tlv_builder_append_short(tlv, - (li->eutran_ci << 4) | 0xf))) && - stk_tlv_builder_close_container(tlv); + if (stk_tlv_builder_open_container(tlv, cr, tag, FALSE) == FALSE) + return FALSE; + + if (stk_tlv_builder_append_bytes(tlv, mccmnc, 3) == FALSE) + return FALSE; + + if (stk_tlv_builder_append_short(tlv, li->lac_tac) == FALSE) + return FALSE; + + if (li->has_ci && stk_tlv_builder_append_short(tlv, li->ci) == FALSE) + return FALSE; + + if (li->has_ext_ci && + stk_tlv_builder_append_short(tlv, li->ext_ci) == FALSE) + return FALSE; + + if (li->has_eutran_ci) { + if (stk_tlv_builder_append_short(tlv, + li->eutran_ci >> 12) == FALSE) + return FALSE; + + if (stk_tlv_builder_append_short(tlv, + (li->eutran_ci << 4) | 0xf) == FALSE) + return FALSE; + } + + return stk_tlv_builder_close_container(tlv); } /* Described in TS 102.223 Section 8.20 |