From 0c306f780fd6a81db213f86eb1d2cac68fa05f48 Mon Sep 17 00:00:00 2001 From: "Enrico Weigelt, metux IT consult" Date: Thu, 12 Sep 2024 10:30:14 +0200 Subject: [PATCH] (!1690) pseudoramiX: fix warning from unused REQUEST() macro calls MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit In the SProc*()s, lots of REQUEST() macro calls aren't needed and causing warnings on unused variables: ../pseudoramiX/pseudoramiX.c: In function ‘SProcPseudoramiXQueryVersion’: ../include/dix.h:66:12: warning: unused variable ‘stuff’ [-Wunused-variable] 66 | type * stuff = (type *)client->requestBuffer; | ^~~~~ ../pseudoramiX/pseudoramiX.c:408:5: note: in expansion of macro ‘REQUEST’ 408 | REQUEST(xPanoramiXQueryVersionReq); | ^~~~~~~ ../pseudoramiX/pseudoramiX.c: In function ‘SProcPseudoramiXGetState’: ../include/dix.h:66:12: warning: unused variable ‘stuff’ [-Wunused-variable] 66 | type * stuff = (type *)client->requestBuffer; | ^~~~~ ../pseudoramiX/pseudoramiX.c:419:5: note: in expansion of macro ‘REQUEST’ 419 | REQUEST(xPanoramiXGetStateReq); | ^~~~~~~ ../pseudoramiX/pseudoramiX.c: In function ‘SProcPseudoramiXGetScreenCount’: ../include/dix.h:66:12: warning: unused variable ‘stuff’ [-Wunused-variable] 66 | type * stuff = (type *)client->requestBuffer; | ^~~~~ ../pseudoramiX/pseudoramiX.c:430:5: note: in expansion of macro ‘REQUEST’ 430 | REQUEST(xPanoramiXGetScreenCountReq); | ^~~~~~~ ../pseudoramiX/pseudoramiX.c: In function ‘SProcPseudoramiXGetScreenSize’: ../include/dix.h:66:12: warning: unused variable ‘stuff’ [-Wunused-variable] 66 | type * stuff = (type *)client->requestBuffer; | ^~~~~ ../pseudoramiX/pseudoramiX.c:441:5: note: in expansion of macro ‘REQUEST’ 441 | REQUEST(xPanoramiXGetScreenSizeReq); | ^~~~~~~ ../pseudoramiX/pseudoramiX.c: In function ‘SProcPseudoramiXIsActive’: ../include/dix.h:66:12: warning: unused variable ‘stuff’ [-Wunused-variable] 66 | type * stuff = (type *)client->requestBuffer; | ^~~~~ ../pseudoramiX/pseudoramiX.c:452:5: note: in expansion of macro ‘REQUEST’ 452 | REQUEST(xXineramaIsActiveReq); | ^~~~~~~ ../pseudoramiX/pseudoramiX.c: In function ‘SProcPseudoramiXQueryScreens’: ../include/dix.h:66:12: warning: unused variable ‘stuff’ [-Wunused-variable] 66 | type * stuff = (type *)client->requestBuffer; | ^~~~~ ../pseudoramiX/pseudoramiX.c:463:5: note: in expansion of macro ‘REQUEST’ 463 | REQUEST(xXineramaQueryScreensReq); | ^~~~~~~ Signed-off-by: Enrico Weigelt, metux IT consult --- pseudoramiX/pseudoramiX.c | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/pseudoramiX/pseudoramiX.c b/pseudoramiX/pseudoramiX.c index e5d68222f..c2089753d 100644 --- a/pseudoramiX/pseudoramiX.c +++ b/pseudoramiX/pseudoramiX.c @@ -405,8 +405,6 @@ ProcPseudoramiXDispatch(ClientPtr client) static int SProcPseudoramiXQueryVersion(ClientPtr client) { - REQUEST(xPanoramiXQueryVersionReq); - TRACE; REQUEST_SIZE_MATCH(xPanoramiXQueryVersionReq); @@ -416,8 +414,6 @@ SProcPseudoramiXQueryVersion(ClientPtr client) static int SProcPseudoramiXGetState(ClientPtr client) { - REQUEST(xPanoramiXGetStateReq); - TRACE; REQUEST_SIZE_MATCH(xPanoramiXGetStateReq); @@ -427,8 +423,6 @@ SProcPseudoramiXGetState(ClientPtr client) static int SProcPseudoramiXGetScreenCount(ClientPtr client) { - REQUEST(xPanoramiXGetScreenCountReq); - TRACE; REQUEST_SIZE_MATCH(xPanoramiXGetScreenCountReq); @@ -438,8 +432,6 @@ SProcPseudoramiXGetScreenCount(ClientPtr client) static int SProcPseudoramiXGetScreenSize(ClientPtr client) { - REQUEST(xPanoramiXGetScreenSizeReq); - TRACE; REQUEST_SIZE_MATCH(xPanoramiXGetScreenSizeReq); @@ -449,8 +441,6 @@ SProcPseudoramiXGetScreenSize(ClientPtr client) static int SProcPseudoramiXIsActive(ClientPtr client) { - REQUEST(xXineramaIsActiveReq); - TRACE; REQUEST_SIZE_MATCH(xXineramaIsActiveReq); @@ -460,8 +450,6 @@ SProcPseudoramiXIsActive(ClientPtr client) static int SProcPseudoramiXQueryScreens(ClientPtr client) { - REQUEST(xXineramaQueryScreensReq); - TRACE; REQUEST_SIZE_MATCH(xXineramaQueryScreensReq);