diff options
| author | Denis Kenzior <denkenz@gmail.com> | 2010-05-28 11:31:26 -0500 |
|---|---|---|
| committer | Denis Kenzior <denkenz@gmail.com> | 2010-05-28 11:32:04 -0500 |
| commit | 3dc3f69b8610266b3484291370161e1728c2e8d1 (patch) | |
| tree | 60d9d12cf960ac630102d5c41820ec79df213aba /src | |
| parent | 88e4f807f86819be4b8c711e3ae3d68c6151acef (diff) | |
| download | ofono-3dc3f69b8610266b3484291370161e1728c2e8d1.tar.bz2 | |
stkutil: Fix style in build_dataobj_timer_value
Diffstat (limited to 'src')
| -rw-r--r-- | src/stkutil.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/src/stkutil.c b/src/stkutil.c index e333e1bd..a4e98f0d 100644 --- a/src/stkutil.c +++ b/src/stkutil.c @@ -3563,14 +3563,16 @@ static gboolean build_dataobj_timer_value(struct stk_tlv_builder *tlv, const struct stk_timer_value *value = data; unsigned char tag = STK_DATA_OBJECT_TYPE_TIMER_VALUE; - return value->has_value == FALSE || - (stk_tlv_builder_open_container(tlv, cr, tag, FALSE) && + if (value->has_value == FALSE) + return TRUE; + #define TO_BCD(bin) ((((bin) / 10) & 0xf) | (((bin) % 10) << 4)) - stk_tlv_builder_append_byte(tlv, TO_BCD(value->hour)) && - stk_tlv_builder_append_byte(tlv, TO_BCD(value->minute)) && - stk_tlv_builder_append_byte(tlv, TO_BCD(value->second)) && + return stk_tlv_builder_open_container(tlv, cr, tag, FALSE) && + stk_tlv_builder_append_byte(tlv, TO_BCD(value->hour)) && + stk_tlv_builder_append_byte(tlv, TO_BCD(value->minute)) && + stk_tlv_builder_append_byte(tlv, TO_BCD(value->second)) && + stk_tlv_builder_close_container(tlv); #undef TO_BCD - stk_tlv_builder_close_container(tlv)); } /* Described in TS 102.223 Section 8.39 */ |