]> de.git.xonotic.org Git - xonotic/darkplaces.git/blobdiff - vid_wgl.c
Cleaned up alot more memory leaks. (still get 720 leaks just running demo1.dem)
[xonotic/darkplaces.git] / vid_wgl.c
index d95526757ee2e0a18d300dd03271ef0aa14b8b8f..b8f3a8c908d1dbfe9fe41ea1fa73c2548dcd16bc 100644 (file)
--- a/vid_wgl.c
+++ b/vid_wgl.c
@@ -73,16 +73,18 @@ static dllfunction_t wglswapintervalfuncs[] =
        {NULL, NULL}
 };
 
-qboolean scr_skipupdate;
-
 static DEVMODE gdevmode;
 static qboolean vid_initialized = false;
 static qboolean vid_wassuspended = false;
-static int vid_usingmouse;
-extern qboolean mouseactive;  // from in_win.c
+static qboolean vid_usingmouse = false;
+static qboolean vid_usingvsync = false;
+static qboolean vid_usemouse = false;
+static qboolean vid_usevsync = false;
 static HICON hIcon;
 
 HWND mainwindow;
+static HDC      baseDC;
+static HGLRC baseRC;
 
 //HWND WINAPI InitializeWindow (HINSTANCE hInstance, int nCmdShow);
 
@@ -145,7 +147,7 @@ HRESULT (WINAPI *pDirectInputCreate)(HINSTANCE hinst, DWORD dwVersion,
 int                    mouse_buttons;
 int                    mouse_oldbuttonstate;
 POINT          current_pos;
-int                    mouse_x, mouse_y, old_mouse_x, old_mouse_y, mx_accum, my_accum;
+int                    mouse_x, mouse_y, old_mouse_x, old_mouse_y;
 
 static qboolean        restore_spi;
 static int             originalmouseparms[3], newmouseparms[3] = {0, 0, 1};
@@ -260,7 +262,7 @@ static DIDATAFORMAT df = {
 // forward-referenced functions
 void IN_StartupJoystick (void);
 void Joy_AdvancedUpdate_f (void);
-void IN_JoyMove (usercmd_t *cmd);
+void IN_JoyMove (void);
 void IN_StartupMouse (void);
 
 /*
@@ -303,25 +305,11 @@ void VID_GetWindowSize (int *x, int *y, int *width, int *height)
 
 void VID_Finish (void)
 {
-       HDC hdc;
-       int vid_usemouse;
-       static int      old_vsync       = -1;
-
-       if (old_vsync != vid_vsync.integer)
+       vid_usevsync = vid_vsync.integer && !cls.timedemo && gl_videosyncavailable;
+       if (vid_usingvsync != vid_usevsync && gl_videosyncavailable)
        {
-               old_vsync = bound(0, vid_vsync.integer, 1);
-               Cvar_SetValueQuick(&vid_vsync, old_vsync);
-               if (gl_videosyncavailable)
-                       qwglSwapIntervalEXT (old_vsync);
-       }
-
-       if (r_render.integer && !scr_skipupdate)
-       {
-               if (r_speeds.integer || gl_finish.integer)
-                       qglFinish();
-               hdc = GetDC(mainwindow);
-               SwapBuffers(hdc);
-               ReleaseDC(mainwindow, hdc);
+               vid_usingvsync = vid_usevsync;
+               qwglSwapIntervalEXT (vid_usevsync);
        }
 
 // handle the mouse state when windowed if that's changed
@@ -350,6 +338,13 @@ void VID_Finish (void)
                        IN_ShowMouse();
                }
        }
+
+       if (r_render.integer && !vid_hidden)
+       {
+               if (r_speeds.integer || gl_finish.integer)
+                       qglFinish();
+               SwapBuffers(baseDC);
+       }
 }
 
 //==========================================================================
@@ -378,7 +373,7 @@ MapKey
 Map from windows to quake keynums
 =======
 */
-int MapKey (int key, int virtualkey)
+static int MapKey (int key, int virtualkey)
 {
        int result;
        int modified = (key >> 16) & 255;
@@ -525,9 +520,6 @@ void Sys_SendKeyEvents (void)
 
        while (PeekMessage (&msg, NULL, 0, 0, PM_NOREMOVE))
        {
-       // we always update if there are any event, even if we're paused
-               scr_skipupdate = 0;
-
                if (!GetMessage (&msg, NULL, 0, 0))
                        Sys_Quit ();
 
@@ -790,7 +782,6 @@ int VID_InitMode (int fullscreen, int width, int height, int bpp)
        };
        int pixelformat;
        DWORD WindowStyle, ExWindowStyle;
-       HGLRC baseRC;
        int CenterX, CenterY;
        const char *gldrivername;
        int depth;
@@ -811,6 +802,7 @@ int VID_InitMode (int fullscreen, int width, int height, int bpp)
        }
 
        gldrivername = "opengl32.dll";
+// COMMANDLINEOPTION: Windows WGL: -gl_driver <drivername> selects a GL driver library, default is opengl32.dll, useful only for 3dfxogl.dll or 3dfxvgl.dll, if you don't know what this is for, you don't need it
        i = COM_CheckParm("-gl_driver");
        if (i && i < com_argc - 1)
                gldrivername = com_argv[i + 1];
@@ -934,19 +926,19 @@ int VID_InitMode (int fullscreen, int width, int height, int bpp)
        // fix the leftover Alt from any Alt-Tab or the like that switched us away
        ClearAllStates ();
 
-       hdc = GetDC(mainwindow);
+       baseDC = GetDC(mainwindow);
 
-       if ((pixelformat = ChoosePixelFormat(hdc, &pfd)) == 0)
+       if ((pixelformat = ChoosePixelFormat(baseDC, &pfd)) == 0)
        {
                VID_Shutdown();
-               Con_Printf("ChoosePixelFormat(%d, %p) failed\n", hdc, &pfd);
+               Con_Printf("ChoosePixelFormat(%d, %p) failed\n", baseDC, &pfd);
                return false;
        }
 
-       if (SetPixelFormat(hdc, pixelformat, &pfd) == false)
+       if (SetPixelFormat(baseDC, pixelformat, &pfd) == false)
        {
                VID_Shutdown();
-               Con_Printf("SetPixelFormat(%d, %d, %p) failed\n", hdc, pixelformat, &pfd);
+               Con_Printf("SetPixelFormat(%d, %d, %p) failed\n", baseDC, pixelformat, &pfd);
                return false;
        }
 
@@ -957,17 +949,17 @@ int VID_InitMode (int fullscreen, int width, int height, int bpp)
                return false;
        }
 
-       baseRC = qwglCreateContext(hdc);
+       baseRC = qwglCreateContext(baseDC);
        if (!baseRC)
        {
                VID_Shutdown();
                Con_Print("Could not initialize GL (wglCreateContext failed).\n\nMake sure you are in 65536 color mode, and try running -window.\n");
                return false;
        }
-       if (!qwglMakeCurrent(hdc, baseRC))
+       if (!qwglMakeCurrent(baseDC, baseRC))
        {
                VID_Shutdown();
-               Con_Printf("wglMakeCurrent(%d, %d) failed\n", hdc, baseRC);
+               Con_Printf("wglMakeCurrent(%d, %d) failed\n", baseDC, baseRC);
                return false;
        }
 
@@ -989,10 +981,11 @@ int VID_InitMode (int fullscreen, int width, int height, int bpp)
        gl_videosyncavailable = false;
 
        if (qwglGetExtensionsStringARB)
-               gl_platformextensions = qwglGetExtensionsStringARB(hdc);
+               gl_platformextensions = qwglGetExtensionsStringARB(baseDC);
 
-       gl_videosyncavailable = GL_CheckExtension("WGL_EXT_swap_control", wglswapintervalfuncs, NULL, false);
-       ReleaseDC(mainwindow, hdc);
+// COMMANDLINEOPTION: Windows WGL: -novideosync disables WGL_EXT_swap_control
+       gl_videosyncavailable = GL_CheckExtension("WGL_EXT_swap_control", wglswapintervalfuncs, "-novideosync", false);
+       //ReleaseDC(mainwindow, hdc);
 
        GL_Init ();
 
@@ -1019,9 +1012,6 @@ int VID_InitMode (int fullscreen, int width, int height, int bpp)
 static void IN_Shutdown(void);
 void VID_Shutdown (void)
 {
-       HGLRC hRC = 0;
-       HDC hDC = 0;
-
        if(vid_initialized == false)
                return;
 
@@ -1029,18 +1019,14 @@ void VID_Shutdown (void)
 
        vid_initialized = false;
        IN_Shutdown();
-       if (qwglGetCurrentContext)
-               hRC = qwglGetCurrentContext();
-       if (qwglGetCurrentDC)
-               hDC = qwglGetCurrentDC();
        if (qwglMakeCurrent)
                qwglMakeCurrent(NULL, NULL);
-       if (hRC && qwglDeleteContext)
-               qwglDeleteContext(hRC);
+       if (baseRC && qwglDeleteContext)
+               qwglDeleteContext(baseRC);
        // close the library before we get rid of the window
        GL_CloseLibrary();
-       if (hDC && mainwindow)
-               ReleaseDC(mainwindow, hDC);
+       if (baseDC && mainwindow)
+               ReleaseDC(mainwindow, baseDC);
        AppActivate(false, false);
        if (mainwindow)
                DestroyWindow(mainwindow);
@@ -1190,7 +1176,7 @@ qboolean IN_InitDInput (void)
 
                if (hInstDI == NULL)
                {
-                       Con_SafePrint("Couldn't load dinput.dll\n");
+                       Con_Print("Couldn't load dinput.dll\n");
                        return false;
                }
        }
@@ -1201,7 +1187,7 @@ qboolean IN_InitDInput (void)
 
                if (!pDirectInputCreate)
                {
-                       Con_SafePrint("Couldn't get DI proc addr\n");
+                       Con_Print("Couldn't get DI proc addr\n");
                        return false;
                }
        }
@@ -1219,7 +1205,7 @@ qboolean IN_InitDInput (void)
 
        if (FAILED(hr))
        {
-               Con_SafePrint("Couldn't open DI mouse device\n");
+               Con_Print("Couldn't open DI mouse device\n");
                return false;
        }
 
@@ -1228,7 +1214,7 @@ qboolean IN_InitDInput (void)
 
        if (FAILED(hr))
        {
-               Con_SafePrint("Couldn't set DI mouse format\n");
+               Con_Print("Couldn't set DI mouse format\n");
                return false;
        }
 
@@ -1238,7 +1224,7 @@ qboolean IN_InitDInput (void)
 
        if (FAILED(hr))
        {
-               Con_SafePrint("Couldn't set DI coop level\n");
+               Con_Print("Couldn't set DI coop level\n");
                return false;
        }
 
@@ -1249,7 +1235,7 @@ qboolean IN_InitDInput (void)
 
        if (FAILED(hr))
        {
-               Con_SafePrint("Couldn't set DI buffersize\n");
+               Con_Print("Couldn't set DI buffersize\n");
                return false;
        }
 
@@ -1269,17 +1255,18 @@ void IN_StartupMouse (void)
 
        mouseinitialized = true;
 
+// COMMANDLINEOPTION: Windows Input: -dinput enables DirectInput for mouse/joystick input
        if (COM_CheckParm ("-dinput"))
        {
                dinput = IN_InitDInput ();
 
                if (dinput)
                {
-                       Con_SafePrint("DirectInput initialized\n");
+                       Con_Print("DirectInput initialized\n");
                }
                else
                {
-                       Con_SafePrint("DirectInput not initialized\n");
+                       Con_Print("DirectInput not initialized\n");
                }
        }
 
@@ -1289,15 +1276,18 @@ void IN_StartupMouse (void)
 
                if (mouseparmsvalid)
                {
+// COMMANDLINEOPTION: Windows GDI Input: -noforcemspd disables setting of mouse speed (not used with -dinput, windows only)
                        if ( COM_CheckParm ("-noforcemspd") )
                                newmouseparms[2] = originalmouseparms[2];
 
+// COMMANDLINEOPTION: Windows GDI Input: -noforcemaccel disables setting of mouse acceleration (not used with -dinput, windows only)
                        if ( COM_CheckParm ("-noforcemaccel") )
                        {
                                newmouseparms[0] = originalmouseparms[0];
                                newmouseparms[1] = originalmouseparms[1];
                        }
 
+// COMMANDLINEOPTION: Windows GDI Input: -noforcemparms disables setting of mouse parameters (not used with -dinput, windows only)
                        if ( COM_CheckParm ("-noforcemparms") )
                        {
                                newmouseparms[0] = originalmouseparms[0];
@@ -1353,7 +1343,7 @@ void IN_MouseEvent (int mstate)
 IN_MouseMove
 ===========
 */
-void IN_MouseMove (usercmd_t *cmd)
+void IN_MouseMove (void)
 {
        int                                     i, mx, my;
        DIDEVICEOBJECTDATA      od;
@@ -1364,7 +1354,7 @@ void IN_MouseMove (usercmd_t *cmd)
        {
                GetCursorPos (&current_pos);
                //ui_mouseupdate(current_pos.x - window_x, current_pos.y - window_y);
-               IN_Mouse( cmd, 0, 0 );
+               IN_Mouse( 0, 0 );
                return;
        }
 
@@ -1447,13 +1437,11 @@ void IN_MouseMove (usercmd_t *cmd)
        else
        {
                GetCursorPos (&current_pos);
-               mx = current_pos.x - window_center_x + mx_accum;
-               my = current_pos.y - window_center_y + my_accum;
-               mx_accum = 0;
-               my_accum = 0;
+               mx = current_pos.x - window_center_x;
+               my = current_pos.y - window_center_y;
        }
 
-       IN_Mouse(cmd, mx, my);
+       IN_Mouse(mx, my);
 
        // if the mouse has moved, force it to the center, so there's room to move
        if (!dinput && (mx || my))
@@ -1466,35 +1454,12 @@ void IN_MouseMove (usercmd_t *cmd)
 IN_Move
 ===========
 */
-void IN_Move (usercmd_t *cmd)
+void IN_Move (void)
 {
        if (vid_activewindow && !vid_hidden)
        {
-               IN_MouseMove (cmd);
-               IN_JoyMove (cmd);
-       }
-}
-
-
-/*
-===========
-IN_Accumulate
-===========
-*/
-void IN_Accumulate (void)
-{
-       if (mouseactive)
-       {
-               if (!dinput)
-               {
-                       GetCursorPos (&current_pos);
-
-                       mx_accum += current_pos.x - window_center_x;
-                       my_accum += current_pos.y - window_center_y;
-
-               // force the mouse to the center, so there's room to move
-                       SetCursorPos (window_center_x, window_center_y);
-               }
+               IN_MouseMove ();
+               IN_JoyMove ();
        }
 }
 
@@ -1507,11 +1472,7 @@ IN_ClearStates
 void IN_ClearStates (void)
 {
        if (mouseactive)
-       {
-               mx_accum = 0;
-               my_accum = 0;
                mouse_oldbuttonstate = 0;
-       }
 }
 
 
@@ -1531,6 +1492,7 @@ void IN_StartupJoystick (void)
        joy_avail = false;
 
        // abort startup if user requests no joystick
+// COMMANDLINEOPTION: Windows Input: -nojoy disables joystick support, may be a small speed increase
        if (COM_CheckParm ("-nojoy") || COM_CheckParm("-safe"))
                return;
 
@@ -1790,7 +1752,7 @@ qboolean IN_ReadJoystick (void)
 IN_JoyMove
 ===========
 */
-void IN_JoyMove (usercmd_t *cmd)
+void IN_JoyMove (void)
 {
        float   speed, aspeed;
        float   fAxisValue, fTemp;
@@ -1885,7 +1847,7 @@ void IN_JoyMove (usercmd_t *cmd)
                                // user wants forward control to be forward control
                                if (fabs(fAxisValue) > joy_forwardthreshold.value)
                                {
-                                       cmd->forwardmove += (fAxisValue * joy_forwardsensitivity.value) * speed * cl_forwardspeed.value;
+                                       cl.cmd.forwardmove += (fAxisValue * joy_forwardsensitivity.value) * speed * cl_forwardspeed.value;
                                }
                        }
                        break;
@@ -1893,7 +1855,7 @@ void IN_JoyMove (usercmd_t *cmd)
                case AxisSide:
                        if (fabs(fAxisValue) > joy_sidethreshold.value)
                        {
-                               cmd->sidemove += (fAxisValue * joy_sidesensitivity.value) * speed * cl_sidespeed.value;
+                               cl.cmd.sidemove += (fAxisValue * joy_sidesensitivity.value) * speed * cl_sidespeed.value;
                        }
                        break;
 
@@ -1903,7 +1865,7 @@ void IN_JoyMove (usercmd_t *cmd)
                                // user wants turn control to become side control
                                if (fabs(fAxisValue) > joy_sidethreshold.value)
                                {
-                                       cmd->sidemove -= (fAxisValue * joy_sidesensitivity.value) * speed * cl_sidespeed.value;
+                                       cl.cmd.sidemove -= (fAxisValue * joy_sidesensitivity.value) * speed * cl_sidespeed.value;
                                }
                        }
                        else