There is no reason to set req->firstKeyCode twice when a client, wishful
for changing keyboard mappings, calls XChangeKeyboardMapping. This patch
fixes the mistake by making no functional changes to libX11 whatsoever.

Signed-off-by: Fernando Carrijo <[email protected]>
---
 src/SetPntMap.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/src/SetPntMap.c b/src/SetPntMap.c
index eaa642d..94ed0c3 100644
--- a/src/SetPntMap.c
+++ b/src/SetPntMap.c
@@ -69,7 +69,6 @@ XChangeKeyboardMapping (
     req->firstKeyCode = first_keycode;
     req->keyCodes = nkeycodes;
     req->keySymsPerKeyCode = keysyms_per_keycode;
-    req->firstKeyCode = first_keycode;
     req->length += nkeycodes * keysyms_per_keycode;
     nbytes = keysyms_per_keycode * nkeycodes * 4;
     Data32 (dpy, (long *)keysyms, nbytes);
-- 
1.6.3.3


_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to