From 7735c4462cb98a400616e207a739a59047b687b2 Mon Sep 17 00:00:00 2001 From: "Enrico Weigelt, metux IT consult" Date: Tue, 9 Jul 2024 16:15:19 +0200 Subject: [PATCH] Xext: xf86bigfont: drop some dead code There's some piece ifdef'ed code that doesn't serve any practical purpose. Instead add a little comment telling why that funny way of dispatching (based on request size) is necessary. Signed-off-by: Enrico Weigelt, metux IT consult Part-of: --- Xext/xf86bigfont.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/Xext/xf86bigfont.c b/Xext/xf86bigfont.c index 44a8e7dd8..7d2bc88d7 100644 --- a/Xext/xf86bigfont.c +++ b/Xext/xf86bigfont.c @@ -339,9 +339,7 @@ ProcXF86BigfontQueryFont(ClientPtr client) CARD16 *pUniqIndex2Index; CARD32 nUniqCharInfos; -#if 0 - REQUEST_SIZE_MATCH(xXF86BigfontQueryFontReq); -#else + /* protocol version is decided based on request packet size */ switch (client->req_len) { case 2: /* client with version 1.0 libX11 */ stuff_flags = (client->local && @@ -353,7 +351,7 @@ ProcXF86BigfontQueryFont(ClientPtr client) default: return BadLength; } -#endif + if (dixLookupFontable(&pFont, stuff->id, client, DixGetAttrAccess) != Success) return BadFont; /* procotol spec says only error is BadFont */