From 3bdacc6e0f3d7acd5039f39bac01ac814e304978 Mon Sep 17 00:00:00 2001 From: "Enrico Weigelt, metux IT consult" Date: Mon, 15 Jul 2024 19:12:01 +0200 Subject: [PATCH] xkb: drop unnecessary assignment of core reply struct fields Those fields are already assigned by X_REPLY_SEND*() macros, thus we don't need to - and should not - assign them explicitly. Signed-off-by: Enrico Weigelt, metux IT consult --- xkb/xkb.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/xkb/xkb.c b/xkb/xkb.c index 72bc44c7df..da90df7309 100644 --- a/xkb/xkb.c +++ b/xkb/xkb.c @@ -4894,7 +4894,6 @@ ProcXkbGetGeometry(ClientPtr client) geom = XkbLookupNamedGeometry(dev, stuff->name, &shouldFree); xkbGetGeometryReply reply = { - .type = X_Reply, .deviceID = dev->id, }; status = XkbComputeGetGeometryReplySize(geom, &reply, stuff->name); @@ -6361,9 +6360,7 @@ ProcXkbGetDeviceInfo(ClientPtr client) nameLen = XkbSizeCountedString(dev->name); xkbGetDeviceInfoReply reply = { - .type = X_Reply, .deviceID = dev->id, - .sequenceNumber = client->sequence, .length = bytes_to_int32(nameLen), .present = wanted, .supported = XkbXI_AllDeviceFeaturesMask,