_XkbFilterDeviceBtn: move variable declarations to match usage scope
The main body of this function is an if { } else if { } pair of blocks. Previously there was int button at the top level scope which is used only in the first block, and a redeclaration of int button inside the second block. Since there's no overlap in the code paths for the two uses of button, move the one from the outer block into the first block to help the programmer more quickly determine they are unrelated usages, and to silence the gcc warning of: xkbActions.c: In function '_XkbFilterDeviceBtn': xkbActions.c:999:6: warning: declaration of 'button' shadows a previous local xkbActions.c:955:6: warning: shadowed declaration is here For consistency, move DeviceIntPtr dev declarations as well that are used in the same way. Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Jamey Sharp <jamey@minilop.net> Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
This commit is contained in:
parent
d8eb9b5faa
commit
41dd7cf673
|
@ -951,13 +951,13 @@ _XkbFilterDeviceBtn( XkbSrvInfoPtr xkbi,
|
|||
unsigned keycode,
|
||||
XkbAction * pAction)
|
||||
{
|
||||
DeviceIntPtr dev;
|
||||
int button;
|
||||
|
||||
if (xkbi->device == inputInfo.keyboard)
|
||||
return 0;
|
||||
|
||||
if (filter->keycode==0) { /* initial press */
|
||||
DeviceIntPtr dev;
|
||||
int button;
|
||||
|
||||
_XkbLookupButtonDevice(&dev, pAction->devbtn.device, serverClient,
|
||||
DixUnknownAccess, &button);
|
||||
if (!dev || !dev->public.on)
|
||||
|
@ -996,7 +996,8 @@ int button;
|
|||
}
|
||||
}
|
||||
else if (filter->keycode==keycode) {
|
||||
int button;
|
||||
DeviceIntPtr dev;
|
||||
int button;
|
||||
|
||||
filter->active= 0;
|
||||
_XkbLookupButtonDevice(&dev, filter->upAction.devbtn.device,
|
||||
|
|
Loading…
Reference in New Issue
Block a user