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

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 <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1600>
This commit is contained in:
Enrico Weigelt, metux IT consult 2024-07-09 16:15:19 +02:00 committed by Marge Bot
parent b0272692a1
commit 7735c4462c
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Xext/xf86bigfont.c
View File

@ -339,9 +339,7 @@ ProcXF86BigfontQueryFont(ClientPtr client)
CARD16 *pUniqIndex2Index; CARD16 *pUniqIndex2Index;
CARD32 nUniqCharInfos; CARD32 nUniqCharInfos;
#if 0 /* protocol version is decided based on request packet size */
REQUEST_SIZE_MATCH(xXF86BigfontQueryFontReq);
#else
switch (client->req_len) { switch (client->req_len) {
case 2: /* client with version 1.0 libX11 */ case 2: /* client with version 1.0 libX11 */
stuff_flags = (client->local && stuff_flags = (client->local &&
@ -353,7 +351,7 @@ ProcXF86BigfontQueryFont(ClientPtr client)
default: default:
return BadLength; return BadLength;
} }
#endif
if (dixLookupFontable(&pFont, stuff->id, client, DixGetAttrAccess) != if (dixLookupFontable(&pFont, stuff->id, client, DixGetAttrAccess) !=
Success) Success)
return BadFont; /* procotol spec says only error is BadFont */ return BadFont; /* procotol spec says only error is BadFont */