Hi Antoine, On 02/25/2011 11:10 AM, Antoine Reversat wrote: > --- > drivers/isimodem/voicecall.c | 11 +++++++---- > 1 files changed, 7 insertions(+), 4 deletions(-) > > diff --git a/drivers/isimodem/voicecall.c b/drivers/isimodem/voicecall.c > index 0a32f27..0ad4dfb 100644 > --- a/drivers/isimodem/voicecall.c > +++ b/drivers/isimodem/voicecall.c > @@ -210,10 +210,13 @@ static void isi_call_any_address_sb_proc(struct > isi_voicecall *ivc, > uint8_t len; > char *addr; > > - if (!g_isi_sb_iter_get_byte(sb, &type, 2) || > - !g_isi_sb_iter_get_byte(sb, &pres, 3) || > - !g_isi_sb_iter_get_byte(sb, &len, 5) || > - !g_isi_sb_iter_get_alpha_tag(sb, &addr, 2 * len, 6)) > + if (!g_isi_sb_iter_get_byte(sb, &type, 2)) > + return; > + if (!g_isi_sb_iter_get_byte(sb, &pres, 3)) > + return; > + if (!g_isi_sb_iter_get_byte(sb, &len, 5)) > + return; > + if (!g_isi_sb_iter_get_alpha_tag(sb, &addr, 2 * len, 6)) > return;
This is indeed the preferred structure, however please note doc/coding-style.txt rule M1. > > call->addr_type = type | 0x80; Regards, -Denis _______________________________________________ ofono mailing list ofono@ofono.org http://lists.ofono.org/listinfo/ofono