diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt index cdffa0377..e00f8c1fd 100644 --- a/src/CMakeLists.txt +++ b/src/CMakeLists.txt @@ -584,13 +584,11 @@ set( PLAT_SDL_SOURCES posix/sdl/i_joystick.cpp posix/sdl/i_main.cpp posix/sdl/i_timer.cpp - posix/sdl/sdlvideo.cpp ) + posix/sdl/sdlvideo.cpp + posix/sdl/sdlglvideo.cpp ) set( PLAT_OSX_SOURCES posix/osx/iwadpicker_cocoa.mm posix/osx/zdoom.icns ) - -# Fixme: This must be adjusted to the new way of doing things: -# sdl/sdlglvideo.cpp set( PLAT_COCOA_SOURCES posix/cocoa/critsec.cpp posix/cocoa/i_input.mm diff --git a/src/sdl/sdlglvideo.cpp b/src/posix/sdl/sdlglvideo.cpp similarity index 83% rename from src/sdl/sdlglvideo.cpp rename to src/posix/sdl/sdlglvideo.cpp index 649989e6d..54506b813 100644 --- a/src/sdl/sdlglvideo.cpp +++ b/src/posix/sdl/sdlglvideo.cpp @@ -1,8 +1,6 @@ // HEADER FILES ------------------------------------------------------------ -#include - #include "doomtype.h" #include "templates.h" @@ -48,6 +46,7 @@ extern IVideo *Video; // extern int vid_renderer; EXTERN_CVAR (Float, Gamma) +EXTERN_CVAR (Int, vid_adapter) EXTERN_CVAR (Int, vid_displaybits) EXTERN_CVAR (Int, vid_renderer) @@ -78,6 +77,7 @@ static MiniModeInfo WinModes[] = { 720, 480 }, // 16:10 { 720, 540 }, { 800, 450 }, // 16:9 + { 800, 480 }, { 800, 500 }, // 16:10 { 800, 600 }, { 848, 480 }, // 16:9 @@ -92,23 +92,33 @@ static MiniModeInfo WinModes[] = { 1152, 720 }, // 16:10 { 1152, 864 }, { 1280, 720 }, // 16:9 + { 1280, 854 }, { 1280, 800 }, // 16:10 { 1280, 960 }, - { 1344, 756 }, // 16:9 + { 1280, 1024 }, // 5:4 { 1360, 768 }, // 16:9 + { 1366, 768 }, { 1400, 787 }, // 16:9 { 1400, 875 }, // 16:10 - { 1440, 900 }, { 1400, 1050 }, + { 1440, 900 }, + { 1440, 960 }, + { 1440, 1080 }, { 1600, 900 }, // 16:9 { 1600, 1000 }, // 16:10 { 1600, 1200 }, - { 1680, 1050 }, // 16:10 - { 1920, 1080 }, // 16:9 - { 1920, 1200 }, // 16:10 - { 2054, 1536 }, - { 2560, 1440 }, // 16:9 - { 2880, 1800 } // 16:10 + { 1920, 1080 }, + { 1920, 1200 }, + { 2048, 1536 }, + { 2560, 1440 }, + { 2560, 1600 }, + { 2560, 2048 }, + { 2880, 1800 }, + { 3200, 1800 }, + { 3840, 2160 }, + { 3840, 2400 }, + { 4096, 2160 }, + { 5120, 2880 } }; // CODE -------------------------------------------------------------------- @@ -116,7 +126,6 @@ static MiniModeInfo WinModes[] = SDLGLVideo::SDLGLVideo (int parm) { IteratorBits = 0; - IteratorFS = false; if( SDL_Init( SDL_INIT_VIDEO ) < 0 ) { fprintf( stderr, "Video initialization failed: %s\n", SDL_GetError( ) ); @@ -136,34 +145,19 @@ void SDLGLVideo::StartModeIterator (int bits, bool fs) { IteratorMode = 0; IteratorBits = bits; - IteratorFS = fs; } bool SDLGLVideo::NextMode (int *width, int *height, bool *letterbox) { if (IteratorBits != 8) return false; - - if (!IteratorFS) + + if ((unsigned)IteratorMode < sizeof(WinModes)/sizeof(WinModes[0])) { - if ((unsigned)IteratorMode < sizeof(WinModes)/sizeof(WinModes[0])) - { - *width = WinModes[IteratorMode].Width; - *height = WinModes[IteratorMode].Height; - ++IteratorMode; - return true; - } - } - else - { - SDL_Rect **modes = SDL_ListModes (NULL, SDL_FULLSCREEN|SDL_HWSURFACE); - if (modes != NULL && modes[IteratorMode] != NULL) - { - *width = modes[IteratorMode]->w; - *height = modes[IteratorMode]->h; - ++IteratorMode; - return true; - } + *width = WinModes[IteratorMode].Width; + *height = WinModes[IteratorMode].Height; + ++IteratorMode; + return true; } return false; } @@ -182,11 +176,11 @@ DFrameBuffer *SDLGLVideo::CreateFrameBuffer (int width, int height, bool fullscr if (fb->Width == width && fb->Height == height) { - bool fsnow = (fb->Screen->flags & SDL_FULLSCREEN) != 0; + bool fsnow = (SDL_GetWindowFlags (fb->Screen) & SDL_WINDOW_FULLSCREEN_DESKTOP) != 0; if (fsnow != fullscreen) { - SDL_WM_ToggleFullScreen (fb->Screen); + SDL_SetWindowFullscreen (fb->Screen, fullscreen ? SDL_WINDOW_FULLSCREEN_DESKTOP : 0); } return old; } @@ -290,7 +284,7 @@ bool SDLGLVideo::SetupPixelFormat(bool allowsoftware, int multisample) SDL_GL_SetAttribute( SDL_GL_ALPHA_SIZE, 8 ); SDL_GL_SetAttribute( SDL_GL_DEPTH_SIZE, 24 ); SDL_GL_SetAttribute( SDL_GL_STENCIL_SIZE, 8 ); -// SDL_GL_SetAttribute( SDL_GL_DOUBLEBUFFER, 1 ); + SDL_GL_SetAttribute( SDL_GL_DOUBLEBUFFER, 1 ); if (multisample > 0) { SDL_GL_SetAttribute( SDL_GL_MULTISAMPLEBUFFERS, 1 ); SDL_GL_SetAttribute( SDL_GL_MULTISAMPLESAMPLES, multisample ); @@ -336,30 +330,37 @@ SDLGLFB::SDLGLFB (void *, int width, int height, int, int, bool fullscreen) return; } - - Screen = SDL_SetVideoMode (width, height, - 32, - SDL_HWSURFACE|SDL_HWPALETTE|SDL_OPENGL | SDL_GL_DOUBLEBUFFER|SDL_ANYFORMAT| - (fullscreen ? SDL_FULLSCREEN : 0)); + FString caption; + caption.Format(GAMESIG " %s (%s)", GetVersionString(), GetGitTime()); + Screen = SDL_CreateWindow (caption, + SDL_WINDOWPOS_UNDEFINED_DISPLAY(vid_adapter), SDL_WINDOWPOS_UNDEFINED_DISPLAY(vid_adapter), + width, height, (fullscreen ? SDL_WINDOW_FULLSCREEN_DESKTOP : 0)|SDL_WINDOW_OPENGL); if (Screen == NULL) return; - m_supportsGamma = -1 != SDL_GetGammaRamp(m_origGamma[0], m_origGamma[1], m_origGamma[2]); - -#if defined(__APPLE__) - // Need to set title here because a window is not created yet when calling the same function from main() - char caption[100]; - mysnprintf(caption, countof(caption), GAMESIG " %s (%s)", GetVersionString(), GetGitTime()); - SDL_WM_SetCaption(caption, NULL); -#endif // __APPLE__ + GLContext = SDL_GL_CreateContext(Screen); + if (GLContext == NULL) + return; + + m_supportsGamma = -1 != SDL_GetWindowGammaRamp(Screen, m_origGamma[0], m_origGamma[1], m_origGamma[2]); } SDLGLFB::~SDLGLFB () { - if (m_supportsGamma) + if (Screen) { - SDL_SetGammaRamp(m_origGamma[0], m_origGamma[1], m_origGamma[2]); + if (m_supportsGamma) + { + SDL_SetWindowGammaRamp(Screen, m_origGamma[0], m_origGamma[1], m_origGamma[2]); + } + + if (GLContext) + { + SDL_GL_DeleteContext(GLContext); + } + + SDL_DestroyWindow(Screen); } } @@ -386,7 +387,7 @@ bool SDLGLFB::CanUpdate () void SDLGLFB::SetGammaTable(WORD *tbl) { - SDL_SetGammaRamp(&tbl[0], &tbl[256], &tbl[512]); + SDL_SetWindowGammaRamp(Screen, &tbl[0], &tbl[256], &tbl[512]); } bool SDLGLFB::Lock(bool buffered) @@ -420,7 +421,7 @@ bool SDLGLFB::IsLocked () bool SDLGLFB::IsFullscreen () { - return (Screen->flags & SDL_FULLSCREEN) != 0; + return (SDL_GetWindowFlags (Screen) & SDL_WINDOW_FULLSCREEN_DESKTOP) != 0; } @@ -443,6 +444,6 @@ void SDLGLFB::NewRefreshRate () void SDLGLFB::SwapBuffers() { - SDL_GL_SwapBuffers (); + SDL_GL_SwapWindow (Screen); } diff --git a/src/sdl/sdlglvideo.h b/src/posix/sdl/sdlglvideo.h similarity index 96% rename from src/sdl/sdlglvideo.h rename to src/posix/sdl/sdlglvideo.h index 205e416c0..c2be3ba47 100644 --- a/src/sdl/sdlglvideo.h +++ b/src/posix/sdl/sdlglvideo.h @@ -3,7 +3,6 @@ #include "hardware.h" #include "v_video.h" -#include #include "gl/system/gl_system.h" EXTERN_CVAR (Float, dimamount) @@ -33,7 +32,6 @@ class SDLGLVideo : public IVideo private: int IteratorMode; int IteratorBits; - bool IteratorFS; }; class SDLGLFB : public DFrameBuffer { @@ -70,9 +68,11 @@ protected: SDLGLFB () {} BYTE GammaTable[3][256]; bool UpdatePending; - - SDL_Surface *Screen; - + + SDL_Window *Screen; + + SDL_GLContext GLContext; + void UpdateColors (); int m_Lock;