Xext: vidmode: use REPLY_*() macros for preparing / sending replies

Use the new macros for preparing and sending replies to clients.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
This commit is contained in:
Enrico Weigelt, metux IT consult
2024-07-10 15:42:24 +02:00
parent 0b3029022b
commit a4142d80ec

View File

@@ -205,10 +205,8 @@ ProcVidModeQueryVersion(ClientPtr client)
.minorVersion = SERVER_XF86VIDMODE_MINOR_VERSION
};
if (client->swapped) {
swaps(&reply.majorVersion);
swaps(&reply.minorVersion);
}
REPLY_FIELD_CARD16(majorVersion);
REPLY_FIELD_CARD16(minorVersion);
return X_SEND_REPLY_SIMPLE(client, reply);
}
@@ -267,20 +265,19 @@ ProcVidModeGetModeLine(ClientPtr client)
reply.vdisplay, reply.vsyncstart, reply.vsyncend,
reply.vtotal, (unsigned long) reply.flags);
if (client->swapped) {
swapl(&reply.dotclock);
swaps(&reply.hdisplay);
swaps(&reply.hsyncstart);
swaps(&reply.hsyncend);
swaps(&reply.htotal);
swaps(&reply.hskew);
swaps(&reply.vdisplay);
swaps(&reply.vsyncstart);
swaps(&reply.vsyncend);
swaps(&reply.vtotal);
swapl(&reply.flags);
swapl(&reply.privsize);
}
REPLY_FIELD_CARD32(dotclock);
REPLY_FIELD_CARD16(hdisplay);
REPLY_FIELD_CARD16(hsyncstart);
REPLY_FIELD_CARD16(hsyncend);
REPLY_FIELD_CARD16(htotal);
REPLY_FIELD_CARD16(hskew);
REPLY_FIELD_CARD16(vdisplay);
REPLY_FIELD_CARD16(vsyncstart);
REPLY_FIELD_CARD16(vsyncend);
REPLY_FIELD_CARD16(vtotal);
REPLY_FIELD_CARD32(flags);
REPLY_FIELD_CARD32(privsize);
if (ver < 2) {
xXF86OldVidModeGetModeLineReply oldrep = {
.dotclock = reply.dotclock,
@@ -379,9 +376,7 @@ ProcVidModeGetAllModeLines(ClientPtr client)
.modecount = modecount
};
if (client->swapped) {
swapl(&reply.modecount);
}
REPLY_FIELD_CARD32(modecount);
return X_SEND_REPLY_WITH_RPCBUF(client, reply, rpcbuf);
}
@@ -1082,12 +1077,11 @@ VidModeValidateModeLine(ClientPtr client, xXF86VidModeValidateModeLineReq *stuff
xXF86VidModeValidateModeLineReply reply = {
.status = status
};
if (client->swapped) {
swapl(&reply.status);
}
DebugF("ValidateModeLine - Succeeded (status = %d)\n", status);
REPLY_FIELD_CARD32(status);
return X_SEND_REPLY_SIMPLE(client, reply);
}
@@ -1351,10 +1345,8 @@ ProcVidModeGetViewPort(ClientPtr client)
.y = y
};
if (client->swapped) {
swapl(&reply.x);
swapl(&reply.y);
}
REPLY_FIELD_CARD32(x);
REPLY_FIELD_CARD32(y);
return X_SEND_REPLY_SIMPLE(client, reply);
}
@@ -1434,11 +1426,9 @@ ProcVidModeGetDotClocks(ClientPtr client)
.flags = (ClockProg ? CLKFLAG_PROGRAMABLE : 0),
};
if (client->swapped) {
swapl(&reply.clocks);
swapl(&reply.maxclocks);
swapl(&reply.flags);
}
REPLY_FIELD_CARD32(clocks);
REPLY_FIELD_CARD32(maxclocks);
REPLY_FIELD_CARD32(flags);
return X_SEND_REPLY_WITH_RPCBUF(client, reply, rpcbuf);
}
@@ -1503,11 +1493,10 @@ ProcVidModeGetGamma(ClientPtr client)
.green = (CARD32) (green * 10000.),
.blue = (CARD32) (blue * 10000.)
};
if (client->swapped) {
swapl(&reply.red);
swapl(&reply.green);
swapl(&reply.blue);
}
REPLY_FIELD_CARD32(red);
REPLY_FIELD_CARD32(green);
REPLY_FIELD_CARD32(blue);
return X_SEND_REPLY_SIMPLE(client, reply);
}
@@ -1596,9 +1585,8 @@ ProcVidModeGetGammaRamp(ClientPtr client)
xXF86VidModeGetGammaRampReply reply = {
.size = stuff->size
};
if (client->swapped) {
swaps(&reply.size);
}
REPLY_FIELD_CARD16(size);
return X_SEND_REPLY_WITH_RPCBUF(client, reply, rpcbuf);
}
@@ -1622,10 +1610,8 @@ ProcVidModeGetGammaRampSize(ClientPtr client)
xXF86VidModeGetGammaRampSizeReply reply = {
.size = pVidMode->GetGammaRampSize(pScreen)
};
if (client->swapped) {
swaps(&reply.size);
}
REPLY_FIELD_CARD16(size);
return X_SEND_REPLY_SIMPLE(client, reply);
}
@@ -1644,9 +1630,7 @@ ProcVidModeGetPermissions(ClientPtr client)
XF86VM_WRITE_PERMISSION : 0)),
};
if (client->swapped) {
swapl(&reply.permissions);
}
REPLY_FIELD_CARD32(permissions);
return X_SEND_REPLY_SIMPLE(client, reply);
}