Re: [Patch] Missing arg for PackBlock in nk6510.c

[Reiner Steib]

On Fri, Jan 30 2004, Pawel Kot wrote:

> On Fri, 30 Jan 2004, Reiner Steib wrote:
>
>> in the current CVS version, common/phones/nk6510.c didn't compile.
>> The attached patch fixes this.  I'm not sure if it's the correct way
>> to fix the problem, so please check.
>
> Correct. My fault, sorry. Commited into cvs.

Thanks.  Was it intended that you changed the second argument for
PackBlock from "j * 2 + 6" to "j * 2 + 5"?

--8<---------------cut here---------------start------------->8---
--- nk6510.c    29 Jan 2004 22:55:50 -0000      1.139
+++ nk6510.c    30 Jan 2004 20:04:02 -0000      1.140
@@ -1,6 +1,6 @@
 /*
 
-  $Id: nk6510.c,v 1.139 2004/01/29 22:55:50 uid66843 Exp $
+  $Id: nk6510.c,v 1.140 2004/01/30 20:04:02 uid66843 Exp $
 
   G N O K I I
 
@@ -1759,7 +1759,7 @@
                        char_unicode_encode((string + 5), entry->number, j);
                        string[j * 2 + 1] = 0;
                        string[4] = j * 2;
-                       count += PackBlock(0x0b, j * 2 + 6, block++, string, 
req + count);
+                       count += PackBlock(0x0b, j * 2 + 5, block++, string, 
req + count, GN_PHONEBOOK_ENTRY_MAX_LENGTH - count);
                } else {
                        /* Default Number */
                        defaultn = 999;
--8<---------------cut here---------------end--------------->8---

Bye, Reiner.
-- 
       ,,,
      (o o)
---ooO-(_)-Ooo--- PGP key available via WWW   http://rsteib.home.pages.de/