GRIM: Check at runtime that shaders are available

engines/grim/gfx_opengl.cpp

@@ -130,8 +130,6 @@ GfxOpenGL::~GfxOpenGL() {
 }
 
 byte *GfxOpenGL::setupScreen(int screenW, int screenH, bool fullscreen) {
-	g_system->setupScreen(screenW, screenH, fullscreen, true);
-
 	_pixelFormat = g_system->getScreenPixelBuffer().getFormat();
 	_screenWidth = screenW;
 	_screenHeight = screenH;

engines/grim/gfx_opengl_shaders.cpp

@@ -405,8 +405,6 @@ void GfxOpenGLS::setupShaders() {
 }
 
 byte *GfxOpenGLS::setupScreen(int screenW, int screenH, bool fullscreen) {
-	g_system->setupScreen(screenW, screenH, fullscreen, true);
-
 	_pixelFormat = g_system->getScreenPixelBuffer().getFormat();
 	_screenWidth = screenW;
 	_screenHeight = screenH;

engines/grim/gfx_tinygl.cpp

@@ -78,8 +78,6 @@ GfxTinyGL::~GfxTinyGL() {
 }
 
 byte *GfxTinyGL::setupScreen(int screenW, int screenH, bool fullscreen) {
-	g_system->setupScreen(screenW, screenH, fullscreen, false);
-
 	Graphics::PixelBuffer buf = g_system->getScreenPixelBuffer();
 	byte *buffer = buf.getRawBuffer();
 

engines/grim/grim.cpp

@@ -36,6 +36,10 @@
 #include "graphics/pixelbuffer.h"
 #include "graphics/renderer.h"
 
+#ifdef USE_OPENGL
+#include "graphics/opengl/context.h"
+#endif
+
 #include "gui/error.h"
 #include "gui/gui-manager.h"
 #include "gui/message.h"
@@ -228,35 +232,50 @@ LuaBase *GrimEngine::createLua() {
 	return new Lua_V1();
 }
 
-GfxBase *GrimEngine::createRenderer() {
+GfxBase *GrimEngine::createRenderer(int screenW, int screenH) {
 	Common::String rendererConfig = ConfMan.get("renderer");
 	Graphics::RendererType desiredRendererType = Graphics::parseRendererTypeCode(rendererConfig);
 	Graphics::RendererType matchingRendererType = Graphics::getBestMatchingAvailableRendererType(desiredRendererType);
 
+	bool fullscreen = ConfMan.getBool("fullscreen");
+	_softRenderer = matchingRendererType == Graphics::kRendererTypeTinyGL;
+	_system->setupScreen(screenW, screenH, fullscreen, !_softRenderer);
+
+#if defined(USE_OPENGL)
+	// Check the OpenGL context actually supports shaders
+	if (matchingRendererType == Graphics::kRendererTypeOpenGLShaders && !OpenGLContext.shadersSupported) {
+		matchingRendererType = Graphics::kRendererTypeOpenGL;
+	}
+#endif
+
 	if (matchingRendererType != desiredRendererType && desiredRendererType != Graphics::kRendererTypeDefault) {
 		// Display a warning if unable to use the desired renderer
 		warning("Unable to create a '%s' renderer", rendererConfig.c_str());
 	}
 
-	_softRenderer = matchingRendererType == Graphics::kRendererTypeTinyGL;
+	GfxBase *renderer = nullptr;
-
 #if defined(USE_GLES2) || defined(USE_OPENGL_SHADERS)
 	if (matchingRendererType == Graphics::kRendererTypeOpenGLShaders) {
-		return CreateGfxOpenGLShader();
+		renderer = CreateGfxOpenGLShader();
 	}
 #endif
 #if defined(USE_OPENGL) && !defined(USE_GLES2)
 	if (matchingRendererType == Graphics::kRendererTypeOpenGL) {
-		return CreateGfxOpenGL();
+		renderer = CreateGfxOpenGL();
 	}
 #endif
 	if (matchingRendererType == Graphics::kRendererTypeTinyGL) {
-		return CreateGfxTinyGL();
+		renderer = CreateGfxTinyGL();
 	}
 
+	if (!renderer) {
 		error("Unable to create a '%s' renderer", rendererConfig.c_str());
 	}
 
+	renderer->setupScreen(screenW, screenH, fullscreen);
+	return renderer;
+}
+
 const char *GrimEngine::getUpdateFilename() {
 	if (!(getGameFlags() & ADGF_DEMO))
 		return "gfupd101.exe";
@@ -326,9 +345,7 @@ Common::Error GrimEngine::run() {
 	}
 	g_sound = new SoundPlayer();
 
-	bool fullscreen = ConfMan.getBool("fullscreen");
+	g_driver = createRenderer(640, 480);
-	g_driver = createRenderer();
-	g_driver->setupScreen(640, 480, fullscreen);
 	g_driver->loadEmergFont();
 
 	if (getGameType() == GType_MONKEY4 && SearchMan.hasFile("AMWI.m4b")) {
@@ -777,8 +794,7 @@ void GrimEngine::mainLoop() {
 			clearPools();
 
 			delete g_driver;
-			g_driver = createRenderer();
+			g_driver = createRenderer(screenWidth, screenHeight);
-			g_driver->setupScreen(screenWidth, screenHeight, fullscreen);
 			savegameRestore();
 
 			if (mode == DrawMode) {

engines/grim/grim.h

@@ -198,7 +198,7 @@ protected:
 	void cameraPostChangeHandle(int num);
 	void buildActiveActorsList();
 	void savegameCallback();
-	GfxBase *createRenderer();
+	GfxBase *createRenderer(int screenW, int screenH);
 	void playAspyrLogo();
 	virtual LuaBase *createLua();
 	virtual void updateNormalMode();