From b8bfb5792a00fdb0a9347dd624938679cf5d6f12 Mon Sep 17 00:00:00 2001 From: "Enrico Weigelt, metux IT consult" Date: Wed, 3 Sep 2025 16:22:19 +0200 Subject: [PATCH] kdrive: KdCursorOffScreen(): move dx and dy into local scope These are just used locally and always assigned at loop body start, so it's cleaner to declare them locally inside the loop body. Signed-off-by: Enrico Weigelt, metux IT consult --- hw/kdrive/src/kinput.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/hw/kdrive/src/kinput.c b/hw/kdrive/src/kinput.c index f8d4054d3b..e12676e69a 100644 --- a/hw/kdrive/src/kinput.c +++ b/hw/kdrive/src/kinput.c @@ -2116,7 +2116,6 @@ KdCursorOffScreen(ScreenPtr *ppScreen, int *x, int *y) { ScreenPtr pScreen = *ppScreen; int n; - int dx, dy; int best_x, best_y; int n_best_x, n_best_y; CARD32 ms; @@ -2140,8 +2139,8 @@ KdCursorOffScreen(ScreenPtr *ppScreen, int *x, int *y) ScreenPtr walkScreen = screenInfo.screens[n]; if (walkScreen == pScreen) continue; - dx = KdScreenOrigin(walkScreen)->x - KdScreenOrigin(pScreen)->x; - dy = KdScreenOrigin(walkScreen)->y - KdScreenOrigin(pScreen)->y; + int dx = KdScreenOrigin(walkScreen)->x - KdScreenOrigin(pScreen)->x; + int dy = KdScreenOrigin(walkScreen)->y - KdScreenOrigin(pScreen)->y; if (*x < 0) { if (dx < 0 && -dx < best_x) { best_x = -dx;