Re: list-modems patch

Hi Alex, > Sometimes i see > > [root@alexander-desktop ofono-0.31]# /usr/lib/ofono/test/list-modems > [ /huawei5 ] > Features = sim > Powered = 1 > Interfaces = org.ofono.Phonebook org.ofono.AudioSettings > org.ofono.VoiceCallManager org.ofono.SimManager > Online = 0 > Model = E1550 > Manufacturer = huawei > Serial = 353142033084081 > Revision = 11.608.12.00.143 > [ org.ofono.Phonebook ] > [ org.ofono.AudioSettings ] > Traceback (most recent call last): > File "/usr/lib/ofono/test/list-modems", line 61, in <module> > print " %s = %s" % (key, val).encode('ascii') > > > Patch > > > > diff --git a/test/list-modems b/test/list-modems > index 557efd5..df1dca8 100755 > --- a/test/list-modems > +++ b/test/list-modems > @@ -58,6 +58,10 @@ for path, properties in modems: > ")" for text, icon in > properties[key] ]) > else: > val = str(properties[key]) > - print " %s = %s" % (key, val) > + try: > + print " %s = %s" % (key, val) > + except: > + continue > + can you at least print the key value. Just not printing that property at all is bad since it is there. Just marking the value as not printable seems to be the better approach. Regards Marcel