Skip to content
Permalink
Browse files

> Hi Ryan,

> I checked the list you linked to, but did not see any reference to whether
> the Win32 window centering bug was fixed... Has that been addressed at all?

> I refer to the bug where, unless one has globally set the
> SDL_WINDOW_CENTERED variable in the OS, only the 1st window that is created

> will be centered. Subsequent calls to SetVideoMode will create a window tha
t
> shares the same upper-left corner location.

I'm working on that now, thanks for reporting it.  What I'm doing is if the
width and height are the same, I'm leaving the window position alone, but if
they're different they will respect the positioning environment variables.
  • Loading branch information
slouken committed Jul 20, 2007
1 parent 6980dd4 commit be1a39634841f11ab93a5da0ddeccb3bcd821cc7
Showing with 4 additions and 1 deletion.
  1. +4 −1 src/video/windib/SDL_dibvideo.c
@@ -468,6 +468,7 @@ SDL_Surface *DIB_SetVideoMode(_THIS, SDL_Surface *current,
int width, int height, int bpp, Uint32 flags)
{
SDL_Surface *video;
int prev_w, prev_h;
Uint32 prev_flags;
DWORD style;
const DWORD directstyle =
@@ -531,6 +532,8 @@ SDL_Surface *DIB_SetVideoMode(_THIS, SDL_Surface *current,

/* Fill in part of the video surface */
prev_flags = video->flags;
prev_w = video->w;
prev_h = video->h;
video->flags = 0; /* Clear flags */
video->w = width;
video->h = height;
@@ -774,7 +777,7 @@ SDL_Surface *DIB_SetVideoMode(_THIS, SDL_Surface *current,
const char *window = NULL;
const char *center = NULL;

if ( !SDL_windowX && !SDL_windowY ) {
if ( video->w != prev_w || video->h != prev_h ) {
window = SDL_getenv("SDL_VIDEO_WINDOW_POS");
center = SDL_getenv("SDL_VIDEO_CENTERED");
if ( window ) {

0 comments on commit be1a396

Please sign in to comment.