[minicom-Bugs][304061] desync in "Screen and keyboard" with minicom-2.2

[minicom-bugs at alioth.debian.org]

Bugs item #304061, was opened at 2006-10-31 01:46
Status: Open
Priority: 3
Submitted By: Mike Frysinger (vapier-guest)
Assigned to: Nobody (None)
Summary: desync in "Screen and keyboard" with minicom-2.2  
Resolution: None
Group: None
Category: None


Initial Comment:
in the "Screen and keyboard" menu with minicom-2.2 (cvs fails as well), pressing the key does not change the proper entry ... minicom-2.1 works fine

for example, the menu looks like:
...
D - Alarm sound            : Yes
E - Foreground Color (menu): WHITE
F - Background Color (menu): BLACK
G - Foreground Color (term): WHITE
H - Background Color (term): BLACK
I - Foreground Color (stat): WHITE
J - Background Color (stat): BLACK
K - History Buffer Size    : 2000
...

pressing 'D' makes the 'E' field change to yes/no while pressing 'J' makes the 'X' field change and so on and so forth ...

----------------------------------------------------------------------

Comment By: Mike Frysinger (vapier-guest)
Date: 2006-10-31 02:02

Message:
i cant seem to post patches ... oh well, you can find the patch i wrote here:
http://sources.gentoo.org/net-dialup/minicom/files/minicom-2.2-one-off.patch

----------------------------------------------------------------------

Comment By: Mike Frysinger (vapier-guest)
Date: 2006-10-31 01:57

Message:
looks like it's just a display bug ... the wlocate() function in src/config.c:doscrkeyb() is called with the last parameter 1 more than it should be

so if the wlocate() calls are updated to 1 less, it seems to work fine:
case 'A':
...
-        wlocate(w, mbslen (command_key) + 1, 1);
+        wlocate(w, mbslen (command_key) + 1, 0);
...

----------------------------------------------------------------------

You can respond by visiting: 
http://alioth.debian.org/tracker/?func=detail&atid=100031&aid=304061&group_id=30018