]> de.git.xonotic.org Git - xonotic/darkplaces.git/blobdiff - vid_glx.c
more fullscreen cleanup
[xonotic/darkplaces.git] / vid_glx.c
index 5cdd71e29bc0e1fc9f6598ce6c6439cc9dea11c9..433fb4ed7db52e5fc2279ec3c16bf8fbc09f463e 100644 (file)
--- a/vid_glx.c
+++ b/vid_glx.c
@@ -114,7 +114,9 @@ static int win_x, win_y;
 
 static XF86VidModeModeInfo init_vidmode, game_vidmode;
 static qboolean vid_isfullscreen = false;
+static qboolean vid_isvidmodefullscreen = false;
 static qboolean vid_isnetwmfullscreen = false;
+static qboolean vid_isoverrideredirect = false;
 
 static Visual *vidx11_visual;
 static Colormap vidx11_colormap;
@@ -291,7 +293,7 @@ void VID_SetMouse(qboolean fullscreengrab, qboolean relative, qboolean hidecurso
                if (fullscreengrab)
                {
                        XGrabPointer(vidx11_display, win,  True, 0, GrabModeAsync, GrabModeAsync, win, None, CurrentTime);
-                       if (vid_grabkeyboard.integer || (vid_isfullscreen && !vid_isnetwmfullscreen))
+                       if (vid_grabkeyboard.integer || vid_isoverrideredirect)
                                XGrabKeyboard(vidx11_display, win, False, GrabModeAsync, GrabModeAsync, CurrentTime);
                }
                else
@@ -473,10 +475,14 @@ static void HandleEvents(void)
                        // window changed size/location
                        win_x = event.xconfigure.x;
                        win_y = event.xconfigure.y;
-                       if(vid_resizable.integer < 2)
+                       if(vid_resizable.integer < 2 || vid_isnetwmfullscreen)
                        {
                                vid.width = event.xconfigure.width;
                                vid.height = event.xconfigure.height;
+                               if(vid_isnetwmfullscreen)
+                                       Con_Printf("NetWM fullscreen: actually using resolution %dx%d\n", vid.width, vid.height);
+                               else
+                                       Con_DPrintf("Updating to ConfigureNotify resolution %dx%d\n", vid.width, vid.height);
                        }
                        break;
                case DestroyNotify:
@@ -489,63 +495,64 @@ static void HandleEvents(void)
                                Sys_Quit(0);
                        break;
                case MapNotify:
-                       if (vid_isfullscreen && !vid_isnetwmfullscreen)
+                       if (vid_isoverrideredirect)
                                break;
                        // window restored
                        vid_hidden = false;
                        VID_RestoreSystemGamma();
 
-                       if(vid_isfullscreen)
+                       if(vid_isvidmodefullscreen)
                        {
                                // set our video mode
                                XF86VidModeSwitchToMode(vidx11_display, vidx11_screen, &game_vidmode);
 
                                // Move the viewport to top left
                                XF86VidModeSetViewPort(vidx11_display, vidx11_screen, 0, 0);
+                       }
 
+                       if(vid_isnetwmfullscreen)
+                       {
                                // make sure it's fullscreen
-                               {
-                                       XEvent event;
-                                       event.type = ClientMessage;
-                                       event.xclient.serial = 0;
-                                       event.xclient.send_event = True;
-                                       event.xclient.message_type = net_wm_state_atom;
-                                       event.xclient.window = win;
-                                       event.xclient.format = 32;
-                                       event.xclient.data.l[0] = 1;
-                                       event.xclient.data.l[1] = net_wm_state_fullscreen_atom;
-                                       event.xclient.data.l[2] = 0;
-                                       event.xclient.data.l[3] = 1;
-                                       event.xclient.data.l[4] = 0;
-                                       XSendEvent(vidx11_display, root, False, SubstructureRedirectMask | SubstructureNotifyMask, &event);
-
-                                       dowarp = true;
-                               }
+                               XEvent event;
+                               event.type = ClientMessage;
+                               event.xclient.serial = 0;
+                               event.xclient.send_event = True;
+                               event.xclient.message_type = net_wm_state_atom;
+                               event.xclient.window = win;
+                               event.xclient.format = 32;
+                               event.xclient.data.l[0] = 1;
+                               event.xclient.data.l[1] = net_wm_state_fullscreen_atom;
+                               event.xclient.data.l[2] = 0;
+                               event.xclient.data.l[3] = 1;
+                               event.xclient.data.l[4] = 0;
+                               XSendEvent(vidx11_display, root, False, SubstructureRedirectMask | SubstructureNotifyMask, &event);
                        }
 
+                       dowarp = true;
+
                        break;
                case UnmapNotify:
-                       if (vid_isfullscreen && !vid_isnetwmfullscreen)
+                       if (vid_isoverrideredirect)
                                break;
                        // window iconified/rolledup/whatever
                        vid_hidden = true;
                        VID_RestoreSystemGamma();
 
-                       if(vid_isfullscreen)
+                       if(vid_isvidmodefullscreen)
                                XF86VidModeSwitchToMode(vidx11_display, vidx11_screen, &init_vidmode);
 
                        break;
                case FocusIn:
-                       if (vid_isfullscreen && !vid_isnetwmfullscreen)
+                       if (vid_isoverrideredirect)
                                break;
                        // window is now the input focus
                        vid_activewindow = true;
                        break;
                case FocusOut:
-                       if (vid_isfullscreen && !vid_isnetwmfullscreen)
+                       if (vid_isoverrideredirect)
                                break;
 
-                       if(vid_isfullscreen && event.xfocus.mode == NotifyNormal)
+                       if(vid_isnetwmfullscreen && event.xfocus.mode == NotifyNormal)
                        {
                                // iconify netwm fullscreen window when it loses focus
                                // when the user selects it in the taskbar, the window manager will map it again and send MapNotify
@@ -640,7 +647,7 @@ void VID_Shutdown(void)
        VID_RestoreSystemGamma();
 
        // FIXME: glXDestroyContext here?
-       if (vid_isfullscreen)
+       if (vid_isvidmodefullscreen)
                XF86VidModeSwitchToMode(vidx11_display, vidx11_screen, &init_vidmode);
        if (win)
                XDestroyWindow(vidx11_display, win);
@@ -649,6 +656,8 @@ void VID_Shutdown(void)
        vid_hidden = true;
        vid_isfullscreen = false;
        vid_isnetwmfullscreen = false;
+       vid_isvidmodefullscreen = false;
+       vid_isoverrideredirect = false;
        vidx11_display = NULL;
        win = 0;
        ctx = NULL;
@@ -750,7 +759,7 @@ void VID_BuildGLXAttrib(int *attrib, qboolean stencil, qboolean stereobuffer, in
        *attrib++ = None;
 }
 
-int VID_InitMode(int fullscreen, int width, int height, int bpp, int refreshrate, int stereobuffer, int samples)
+int VID_InitMode(int fullscreen, int *width, int *height, int bpp, int refreshrate, int stereobuffer, int samples)
 {
        int i;
        int attrib[32];
@@ -763,6 +772,11 @@ int VID_InitMode(int fullscreen, int width, int height, int bpp, int refreshrate
        int MajorVersion, MinorVersion;
        const char *drivername;
 
+       vid_isfullscreen = false;
+       vid_isnetwmfullscreen = false;
+       vid_isvidmodefullscreen = false;
+       vid_isoverrideredirect = false;
+
 #if defined(__APPLE__) && defined(__MACH__)
        drivername = "/usr/X11R6/lib/libGL.1.dylib";
 #else
@@ -828,13 +842,22 @@ int VID_InitMode(int fullscreen, int width, int height, int bpp, int refreshrate
                return false;
        }
 
-       if (vidmode_ext)
+       if (fullscreen)
        {
-               int best_fit, best_dist, dist, x, y;
+               if(vid_netwmfullscreen.integer)
+               {
+                       // TODO detect WM support
+                       vid_isnetwmfullscreen = true;
+                       vid_isfullscreen = true;
+                       // width and height will be filled in later
+                       Con_DPrintf("Using NetWM fullscreen mode\n");
+               }
 
-               // Are we going fullscreen?  If so, let's change video mode
-               if (fullscreen)
+               if(!vid_isfullscreen && vidmode_ext)
                {
+                       int best_fit, best_dist, dist, x, y;
+
+                       // Are we going fullscreen?  If so, let's change video mode
                        XF86VidModeModeLine *current_vidmode;
                        XF86VidModeModeInfo **vidmodes;
                        int num_vidmodes;
@@ -849,11 +872,11 @@ int VID_InitMode(int fullscreen, int width, int height, int bpp, int refreshrate
 
                        for (i = 0; i < num_vidmodes; i++)
                        {
-                               if (width > vidmodes[i]->hdisplay || height > vidmodes[i]->vdisplay)
+                               if (*width > vidmodes[i]->hdisplay || *height > vidmodes[i]->vdisplay)
                                        continue;
 
-                               x = width - vidmodes[i]->hdisplay;
-                               y = height - vidmodes[i]->vdisplay;
+                               x = *width - vidmodes[i]->hdisplay;
+                               y = *height - vidmodes[i]->vdisplay;
                                dist = (x * x) + (y * y);
                                if (best_fit == -1 || dist < best_dist)
                                {
@@ -867,22 +890,33 @@ int VID_InitMode(int fullscreen, int width, int height, int bpp, int refreshrate
                                // LordHavoc: changed from ActualWidth/ActualHeight =,
                                // to width/height =, so the window will take the full area of
                                // the mode chosen
-                               width = vidmodes[best_fit]->hdisplay;
-                               height = vidmodes[best_fit]->vdisplay;
+                               *width = vidmodes[best_fit]->hdisplay;
+                               *height = vidmodes[best_fit]->vdisplay;
 
                                // change to the mode
                                XF86VidModeSwitchToMode(vidx11_display, vidx11_screen, vidmodes[best_fit]);
                                memcpy(&game_vidmode, vidmodes[best_fit], sizeof(game_vidmode));
+                               vid_isvidmodefullscreen = true;
                                vid_isfullscreen = true;
 
                                // Move the viewport to top left
                                XF86VidModeSetViewPort(vidx11_display, vidx11_screen, 0, 0);
+                               Con_DPrintf("Using XVidMode fullscreen mode at %dx%d\n", *width, *height);
                        }
-                       else
-                               fullscreen = 0;
 
                        free(vidmodes);
                }
+
+               if(!vid_isfullscreen)
+               {
+                       // sorry, no FS available
+                       // use the full desktop resolution
+                       vid_isfullscreen = true;
+                       // width and height will be filled in later
+                       *width = DisplayWidth(vidx11_display, vidx11_screen);
+                       *height = DisplayHeight(vidx11_display, vidx11_screen);
+                       Con_DPrintf("Using X11 fullscreen mode at %dx%d\n", *width, *height);
+               }
        }
 
        // LordHavoc: save the visual for use in gamma ramp settings later
@@ -895,14 +929,13 @@ int VID_InitMode(int fullscreen, int width, int height, int bpp, int refreshrate
        vidx11_colormap = attr.colormap = XCreateColormap(vidx11_display, root, visinfo->visual, AllocNone);
        attr.event_mask = X_MASK;
 
-       if (vid_isfullscreen)
+       if (fullscreen)
        {
-               if(vid_netwmfullscreen.integer)
+               if(vid_isnetwmfullscreen)
                {
                        mask = CWBackPixel | CWColormap | CWSaveUnder | CWBackingStore | CWEventMask;
                        attr.backing_store = NotUseful;
                        attr.save_under = False;
-                       vid_isnetwmfullscreen = true;
                }
                else
                {
@@ -910,16 +943,15 @@ int VID_InitMode(int fullscreen, int width, int height, int bpp, int refreshrate
                        attr.override_redirect = True;
                        attr.backing_store = NotUseful;
                        attr.save_under = False;
-                       vid_isnetwmfullscreen = false;
+                       vid_isoverrideredirect = true; // so it knows to grab
                }
        }
        else
        {
                mask = CWBackPixel | CWBorderPixel | CWColormap | CWEventMask;
-               vid_isnetwmfullscreen = false;
        }
 
-       win = XCreateWindow(vidx11_display, root, 0, 0, width, height, 0, visinfo->depth, InputOutput, visinfo->visual, mask, &attr);
+       win = XCreateWindow(vidx11_display, root, 0, 0, *width, *height, 0, visinfo->depth, InputOutput, visinfo->visual, mask, &attr);
 
        wmhints = XAllocWMHints();
        if(XpmCreatePixmapFromData(vidx11_display, win,
@@ -934,8 +966,8 @@ int VID_InitMode(int fullscreen, int width, int height, int bpp, int refreshrate
        szhints = XAllocSizeHints();
        if(vid_resizable.integer == 0 && !vid_isnetwmfullscreen)
        {
-               szhints->min_width = szhints->max_width = width;
-               szhints->min_height = szhints->max_height = height;
+               szhints->min_width = szhints->max_width = *width;
+               szhints->min_height = szhints->max_height = *height;
                szhints->flags |= PMinSize | PMaxSize;
        }
 
@@ -949,12 +981,16 @@ int VID_InitMode(int fullscreen, int width, int height, int bpp, int refreshrate
 
        XSetWMProtocols(vidx11_display, win, &wm_delete_window_atom, 1);
 
-       if (vid_isfullscreen && !vid_isnetwmfullscreen)
+       if (vid_isoverrideredirect)
        {
                XMoveWindow(vidx11_display, win, 0, 0);
                XRaiseWindow(vidx11_display, win);
                XWarpPointer(vidx11_display, None, win, 0, 0, 0, 0, 0, 0);
                XFlush(vidx11_display);
+       }
+
+       if(vid_isvidmodefullscreen)
+       {
                // Move the viewport to top left
                XF86VidModeSetViewPort(vidx11_display, vidx11_screen, 0, 0);
        }
@@ -1009,6 +1045,7 @@ int VID_InitMode(int fullscreen, int width, int height, int bpp, int refreshrate
        if (!vid_x11_dgasupported)
                Con_Print( "Failed to detect XF86DGA Mouse extension\n" );
 #endif
+
        GL_Init();
        return true;
 }