Re: [PATCH] ModemManager: Change length input to gsm7 to take the number of characters rather than octets

2011-04-14 Thread Dan Williams
On Fri, 2011-04-08 at 12:00 -0400, Nathan Williams wrote: > Sure, updated patch attached. Pushed, thanks. Dan > > - Nathan > > On Thu, Apr 7, 2011 at 6:45 PM, Dan Williams wrote: > On Thu, 2011-04-07 at 18:09 -0400, Nathan Williams wrote: > > As it says in the patch, this

Re: [PATCH] ModemManager: Change length input to gsm7 to take the number of characters rather than octets

2011-04-08 Thread Nathan Williams
Sure, updated patch attached. - Nathan On Thu, Apr 7, 2011 at 6:45 PM, Dan Williams wrote: > On Thu, 2011-04-07 at 18:09 -0400, Nathan Williams wrote: > > As it says in the patch, this is necessary to distinguish between 7- > > and 8-character messages, since both are encoded in 7 octets. >

Re: [PATCH] ModemManager: Change length input to gsm7 to take the number of characters rather than octets

2011-04-07 Thread Dan Williams
On Thu, 2011-04-07 at 17:45 -0500, Dan Williams wrote: > On Thu, 2011-04-07 at 18:09 -0400, Nathan Williams wrote: > > As it says in the patch, this is necessary to distinguish between 7- > > and 8-character messages, since both are encoded in 7 octets. > > Can you fix up the testcases too? > > s

Re: [PATCH] ModemManager: Change length input to gsm7 to take the number of characters rather than octets

2011-04-07 Thread Dan Williams
On Thu, 2011-04-07 at 18:09 -0400, Nathan Williams wrote: > As it says in the patch, this is necessary to distinguish between 7- > and 8-character messages, since both are encoded in 7 octets. Can you fix up the testcases too? src/tests/test_unpack_gsm7: ** ERROR:test-charsets.c:103:test_unpack_g

[PATCH] ModemManager: Change length input to gsm7 to take the number of characters rather than octets

2011-04-07 Thread Nathan Williams
As it says in the patch, this is necessary to distinguish between 7- and 8-character messages, since both are encoded in 7 octets. - Nathan mm-patch-gsm7 Description: Binary data ___ networkmanager-list mailing list networkmanager-list@gnome.org ht