Skip to content
Permalink
Browse files

- implemented OpenGL / Vulkan switch in SDL backend

  • Loading branch information...
alexey-lysiuk committed Mar 12, 2019
1 parent aa84f7b commit 7b5eedea3261abdc8925fb3a3806d5f4627a7122
Showing with 23 additions and 22 deletions.
  1. +13 −13 src/posix/sdl/sdlglvideo.cpp
  2. +9 −0 src/v_video.cpp
  3. +1 −9 src/win32/hardware.cpp
@@ -72,6 +72,7 @@ EXTERN_CVAR (Int, vid_displaybits)
EXTERN_CVAR (Int, vid_maxfps)
EXTERN_CVAR (Int, vid_defwidth)
EXTERN_CVAR (Int, vid_defheight)
EXTERN_CVAR (Int, vid_backend)
EXTERN_CVAR (Bool, cl_capfps)

// PUBLIC DATA DEFINITIONS -------------------------------------------------
@@ -111,7 +112,7 @@ namespace Priv
static const int MIN_HEIGHT = 200;

SDL_Window *window;
bool vulkanSupported;
bool vulkanEnabled;
bool fullscreenSwitch;

void CreateWindow(uint32_t extraFlags)
@@ -197,25 +198,23 @@ class SDLVideo : public IVideo

void I_GetVulkanDrawableSize(int *width, int *height)
{
assert(Priv::vulkanSupported);
assert(Priv::vulkanEnabled);
assert(Priv::window != nullptr);
assert(Priv::Vulkan_GetDrawableSize);
Priv::Vulkan_GetDrawableSize(Priv::window, width, height);
}

bool I_GetVulkanPlatformExtensions(unsigned int *count, const char **names)
{
assert(Priv::vulkanSupported);
assert(Priv::vulkanEnabled);
assert(Priv::window != nullptr);
assert(Priv::Vulkan_GetInstanceExtensions);
return Priv::Vulkan_GetInstanceExtensions(Priv::window, count, names) == SDL_TRUE;
}

bool I_CreateVulkanSurface(VkInstance instance, VkSurfaceKHR *surface)
{
assert(Priv::vulkanSupported);
assert(Priv::vulkanEnabled);
assert(Priv::window != nullptr);
assert(Priv::Vulkan_CreateSurface);
return Priv::Vulkan_CreateSurface(Priv::window, instance, surface) == SDL_TRUE;
}

@@ -234,15 +233,16 @@ SDLVideo::SDLVideo ()
Priv::library.Load({ "libSDL2.so", "libSDL2-2.0.so" });
}

Priv::vulkanSupported = Priv::Vulkan_GetDrawableSize && Priv::Vulkan_GetInstanceExtensions && Priv::Vulkan_CreateSurface;
Priv::vulkanEnabled = vid_backend == 0
&& Priv::Vulkan_GetDrawableSize && Priv::Vulkan_GetInstanceExtensions && Priv::Vulkan_CreateSurface;

if (Priv::vulkanSupported)
if (Priv::vulkanEnabled)
{
Priv::CreateWindow(Priv::VulkanWindowFlag | SDL_WINDOW_HIDDEN);

if (Priv::window == nullptr)
{
Priv::vulkanSupported = false;
Priv::vulkanEnabled = false;
}
}
}
@@ -257,7 +257,7 @@ DFrameBuffer *SDLVideo::CreateFrameBuffer ()
SystemBaseFrameBuffer *fb = nullptr;

// first try Vulkan, if that fails OpenGL
if (Priv::vulkanSupported)
if (Priv::vulkanEnabled)
{
try
{
@@ -267,7 +267,7 @@ DFrameBuffer *SDLVideo::CreateFrameBuffer ()
}
catch (CRecoverableError const&)
{
Priv::vulkanSupported = false;
Priv::vulkanEnabled = false;
}
}

@@ -302,7 +302,7 @@ int SystemBaseFrameBuffer::GetClientWidth()
{
int width = 0;

assert(Priv::vulkanSupported);
assert(Priv::vulkanEnabled);
Priv::Vulkan_GetDrawableSize(Priv::window, &width, nullptr);

return width;
@@ -312,7 +312,7 @@ int SystemBaseFrameBuffer::GetClientHeight()
{
int height = 0;

assert(Priv::vulkanSupported);
assert(Priv::vulkanEnabled);
Priv::Vulkan_GetDrawableSize(Priv::window, nullptr, &height);

return height;
@@ -113,6 +113,15 @@ CUSTOM_CVAR(Int, vid_rendermode, 4, CVAR_ARCHIVE | CVAR_GLOBALCONFIG | CVAR_NOIN
// No further checks needed. All this changes now is which scene drawer the render backend calls.
}

CUSTOM_CVAR(Int, vid_backend, 0, CVAR_ARCHIVE | CVAR_GLOBALCONFIG | CVAR_NOINITCALL)
{
// [SP] This may seem pointless - but I don't want to implement live switching just
// yet - I'm pretty sure it's going to require a lot of reinits and destructions to
// do it right without memory leaks

Printf("Changing the video backend requires a restart for " GAMENAME ".\n");
}

CVAR(Int, vid_renderer, 1, 0) // for some stupid mods which threw caution out of the window...


@@ -50,15 +50,7 @@
#include "swrenderer/r_swrenderer.h"

EXTERN_CVAR(Int, vid_maxfps)

CUSTOM_CVAR(Int, vid_backend, 0, CVAR_ARCHIVE|CVAR_GLOBALCONFIG|CVAR_NOINITCALL)
{
// [SP] This may seem pointless - but I don't want to implement live switching just
// yet - I'm pretty sure it's going to require a lot of reinits and destructions to
// do it right without memory leaks

Printf("Changing the video backend requires a restart for " GAMENAME ".\n");
}
EXTERN_CVAR(Int, vid_backend)

extern HWND Window;

0 comments on commit 7b5eede

Please sign in to comment.
You can’t perform that action at this time.