Navigation Menu

Skip to content

Commit

Permalink
x11: Don't delay an extra 10ms if we were just going to break out of …
Browse files Browse the repository at this point in the history
…the loop.
  • Loading branch information
icculus committed Feb 17, 2020
1 parent 367a8b9 commit e731522
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions src/video/x11/SDL_x11window.c
Expand Up @@ -824,7 +824,7 @@ X11_SetWindowPosition(_THIS, SDL_Window * window)
/* Wait a brief time to see if the window manager decided to let this move happen.
If the window changes at all, even to an unexpected value, we break out. */
timeout = SDL_GetTicks() + 100;
do {
while (SDL_TRUE) {
int x, y;
X11_XSync(display, False);
X11_XGetWindowAttributes(display, data->xwindow, &attrs);
Expand All @@ -839,8 +839,12 @@ X11_SetWindowPosition(_THIS, SDL_Window * window)
break; /* we're at the place we wanted to be anyhow, drop out. */
}

if (SDL_TICKS_PASSED(SDL_GetTicks(), timeout)) {
break;
}

SDL_Delay(10);
} while (!SDL_TICKS_PASSED(SDL_GetTicks(), timeout));
}
}

void
Expand Down

0 comments on commit e731522

Please sign in to comment.