From 99590abe5256a869b281da54889c27c1715dad5b Mon Sep 17 00:00:00 2001 From: "Enrico Weigelt, metux IT consult" Date: Wed, 29 Oct 2025 12:28:27 +0100 Subject: [PATCH] render: PanoramiXRenderComposite() drop `stuff` pointer This pointer just had been kept in at previous commit for keeping the diff small and so easier to review. Now accessing the fields within the local struct directly, dropping the extra pointer. Signed-off-by: Enrico Weigelt, metux IT consult --- render/render.c | 21 ++++++++++----------- 1 file changed, 10 insertions(+), 11 deletions(-) diff --git a/render/render.c b/render/render.c index 5baf302593..b1c0730f62 100644 --- a/render/render.c +++ b/render/render.c @@ -2108,27 +2108,26 @@ PanoramiXRenderComposite(ClientPtr client, xRenderCompositeReq *orig_req) VERIFY_XIN_PICTURE(dst, orig.dst, client, DixWriteAccess); xRenderCompositeReq sub_req = orig; - xRenderCompositeReq *stuff = &sub_req; XINERAMA_FOR_EACH_SCREEN_FORWARD({ - stuff->src = src->info[walkScreenIdx].id; + sub_req.src = src->info[walkScreenIdx].id; if (src->u.pict.root) { - stuff->xSrc = orig.xSrc - walkScreen->x; - stuff->ySrc = orig.ySrc - walkScreen->y; + sub_req.xSrc = orig.xSrc - walkScreen->x; + sub_req.ySrc = orig.ySrc - walkScreen->y; } - stuff->dst = dst->info[walkScreenIdx].id; + sub_req.dst = dst->info[walkScreenIdx].id; if (dst->u.pict.root) { - stuff->xDst = orig.xDst - walkScreen->x; - stuff->yDst = orig.yDst - walkScreen->y; + sub_req.xDst = orig.xDst - walkScreen->x; + sub_req.yDst = orig.yDst - walkScreen->y; } if (msk) { - stuff->mask = msk->info[walkScreenIdx].id; + sub_req.mask = msk->info[walkScreenIdx].id; if (msk->u.pict.root) { - stuff->xMask = orig.xMask - walkScreen->x; - stuff->yMask = orig.yMask - walkScreen->y; + sub_req.xMask = orig.xMask - walkScreen->x; + sub_req.yMask = orig.yMask - walkScreen->y; } } - result = SingleRenderComposite(client, stuff); + result = SingleRenderComposite(client, &sub_req); if (result != Success) break; });