On Wed, Mar 17, 2010 at 11:53:16PM -0300, Fernando Carrijo wrote: > 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
thanks, applied and pushed. Cheers, Peter _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
