frederik
/
xserver
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

(1886) XI: drop extra temp variable in ProcXGetDeviceMotionEvents() 

The value is only checked once, so no need to copy it to a temporary
variable first.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
This commit is contained in:
Enrico Weigelt, metux IT consult 2025-04-01 16:06:49 +02:00
parent cb8df6ac88
commit 417b860a34
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
Xi/gtmotion.c
View File

@ -87,7 +87,7 @@ int
ProcXGetDeviceMotionEvents(ClientPtr client)
{
INT32 *coords = NULL;
int rc, num_events, axes, size = 0;
int rc, axes, size = 0;
DeviceIntPtr dev;
TimeStamp start, stop;
ValuatorClassPtr v;
@ -124,8 +124,7 @@ ProcXGetDeviceMotionEvents(ClientPtr client)
}
if (CompareTimeStamps(stop, currentTime) == LATER)
stop = currentTime;
num_events = v->numMotionEvents;
if (num_events) {
if (v->numMotionEvents) {
size = sizeof(Time) + (axes * sizeof(INT32));
rep.nEvents = GetMotionHistory(dev, (xTimecoord **) &coords, /* XXX */
start.milliseconds, stop.milliseconds,