Extend the ref import API with Vid_RequestRestart().

This function will be used to replace the vid_fullscreen->modified
mechanism used the communicate renderer configuration restarts to the
client with a proper proper API. The implementation is backward
compatible, existing renderers are still working.
This commit is contained in:
Yamagi 2021-04-10 18:26:10 +02:00
parent 886ce1525e
commit 09d645ab4b
2 changed files with 50 additions and 11 deletions

View file

@ -116,6 +116,14 @@ typedef struct {
particle_t *particles;
} refdef_t;
// Renderer restart type.
typedef enum {
RESTART_UNDEF,
RESTART_NO,
RESTART_FULL,
RESTART_PARTIAL
} ref_restart_t;
// FIXME: bump API_VERSION?
#define API_VERSION 5
#define EXPORT
@ -233,6 +241,8 @@ typedef struct
qboolean (IMPORT *GLimp_InitGraphics)(int fullscreen, int *pwidth, int *pheight);
qboolean (IMPORT *GLimp_GetDesktopMode)(int *pwidth, int *pheight);
void (IMPORT *Vid_RequestRestart)(ref_restart_t rs);
} refimport_t;
// this is the only function actually exported at the linker level

View file

@ -300,13 +300,30 @@ void *reflib_handle = NULL;
// Is a renderer loaded and active?
qboolean ref_active = false;
// Renderer restart type requested.
ref_restart_t restart_state = RESTART_UNDEF;
/*
* Called by the renderer to request a restart.
*/
void
VID_RequestRestart(ref_restart_t rs)
{
restart_state = rs;
}
/*
* Restarts the renderer.
*/
void
VID_Restart_f(void)
{
if (restart_state == RESTART_UNDEF)
{
vid_fullscreen->modified = true;
} else {
restart_state = RESTART_FULL;
}
}
/*
@ -392,6 +409,8 @@ VID_LoadRenderer(void)
ri.Vid_GetModeInfo = VID_GetModeInfo;
ri.Vid_MenuInit = VID_MenuInit;
ri.Vid_WriteScreenshot = VID_WriteScreenshot;
ri.Vid_WriteScreenshot = VID_WriteScreenshot;
ri.Vid_RequestRestart = VID_RequestRestart;
// Exchange our export struct with the renderers import struct.
re = GetRefAPI(ri);
@ -436,11 +455,25 @@ VID_LoadRenderer(void)
void
VID_CheckChanges(void)
{
// FIXME: Not with vid_fullscreen, should be a dedicated variable.
// Sounds easy but this vid_fullscreen hack is really messy and
// interacts with several critical places in both the client and
// the renderers...
// Hack around renderers that still abuse vid_fullscreen
// to communicate restart requests to the client.
ref_restart_t rs;
if (restart_state == RESTART_UNDEF)
{
if (vid_fullscreen->modified)
{
rs = RESTART_FULL;
vid_fullscreen->modified = false;
} else {
rs = RESTART_NO;
}
} else {
rs = restart_state;
restart_state = RESTART_NO;
}
if (rs == RESTART_FULL)
{
// Stop sound, because the clients blocks while
// we're reloading the renderer. The sound system
@ -486,16 +519,12 @@ VID_CheckChanges(void)
}
}
// Ignore possible changes in vid_renderer above.
vid_renderer->modified = false;
// Unblock the client.
cls.disable_screen = false;
}
if (vid_renderer->modified)
if (rs == RESTART_PARTIAL)
{
vid_renderer->modified = false;
cl.refresh_prepped = false;
}
}