XLibre/xserver
1
1
Fork 0
mirror of https://github.com/X11Libre/xserver.git synced 2026-03-24 08:04:30 +00:00
Code Issues Packages Projects Releases Wiki Activity

dix: canonical walkScreenIdx variable on screen list iterations

Browse Source 
When iterating screen lists, consistently use the same variable name
`walkScreenIdx` for holding current screen index everywhere.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
This commit is contained in:
Enrico Weigelt, metux IT consult
2025-08-15 14:23:35 +02:00
committed by Enrico Weigelt
parent 4d66d6f9ac
commit e268101b64
11 changed files with 140 additions and 130 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
dix/dispatch.c
View File

@@ -650,10 +650,10 @@ CreateConnectionBlock(void)
connBlockScreenStart = sizesofar;
memset(&depth, 0, sizeof(xDepth));
memset(&visual, 0, sizeof(xVisualType));
for (int i = 0; i < screenInfo.numScreens; i++) {
for (unsigned int walkScreenIdx = 0; walkScreenIdx < screenInfo.numScreens; walkScreenIdx++) {
DepthPtr pDepth;
VisualPtr pVisual;
ScreenPtr walkScreen = screenInfo.screens[i];
ScreenPtr walkScreen = screenInfo.screens[walkScreenIdx];
root.windowId = walkScreen->root->drawable.id;
root.defaultColormap = walkScreen->defColormap;
@@ -3162,8 +3162,8 @@ ProcSetScreenSaver(ClientPtr client)
REQUEST(xSetScreenSaverReq);
REQUEST_SIZE_MATCH(xSetScreenSaverReq);
for (int i = 0; i < screenInfo.numScreens; i++) {
ScreenPtr walkScreen = screenInfo.screens[i];
for (unsigned int walkScreenIdx = 0; walkScreenIdx < screenInfo.numScreens; walkScreenIdx++) {
ScreenPtr walkScreen = screenInfo.screens[walkScreenIdx];
int rc = XaceHookScreensaverAccess(client, walkScreen, DixSetAttrAccess);
if (rc != Success)
return rc;
@@ -3219,8 +3219,8 @@ ProcGetScreenSaver(ClientPtr client)
{
REQUEST_SIZE_MATCH(xReq);
for (int i = 0; i < screenInfo.numScreens; i++) {
ScreenPtr walkScreen = screenInfo.screens[i];
for (unsigned int walkScreenIdx = 0; walkScreenIdx < screenInfo.numScreens; walkScreenIdx++) {
ScreenPtr walkScreen = screenInfo.screens[walkScreenIdx];
int rc = XaceHookScreensaverAccess(client, walkScreen, DixGetAttrAccess);
if (rc != Success)
return rc;
@@ -3728,8 +3728,8 @@ SendConnSetup(ClientPtr client, const char *reason)
numScreens = ((xConnSetup *) ConnectionInfo)->numRoots;
#endif /* XINERAMA */
for (int i = 0; i < numScreens; i++) {
ScreenPtr walkScreen = screenInfo.screens[i];
for (unsigned int walkScreenIdx = 0; walkScreenIdx < numScreens; walkScreenIdx++) {
ScreenPtr walkScreen = screenInfo.screens[walkScreenIdx];
xDepth *pDepth;
WindowPtr pRoot = walkScreen->root;