changeset 800:ec23202df6a6

Minor cleanup
author Michael Pavone <pavone@retrodev.com>
date Sun, 26 Jul 2015 13:33:48 -0700
parents 0b692b5d154b
children 092524bb2e8f
files render_sdl.c
diffstat 1 files changed, 8 insertions(+), 12 deletions(-) [+]
line wrap: on
line diff
--- a/render_sdl.c	Sun Jul 26 13:25:31 2015 -0700
+++ b/render_sdl.c	Sun Jul 26 13:33:48 2015 -0700
@@ -126,14 +126,14 @@
 		f = fopen(shader_path, "r");
 		free(shader_path);
 		if (!f) {
-			fprintf(stderr, "Failed to open shader file %s for reading\n", fname);
+			warning("Failed to open shader file %s for reading\n", fname);
 			return 0;
 		}
 	}
 	long fsize = file_size(f);
 	GLchar * text = malloc(fsize);
 	if (fread(text, 1, fsize, f) != fsize) {
-		fprintf(stderr, "Error reading from shader file %s\n", fname);
+		warning("Error reading from shader file %s\n", fname);
 		free(text);
 		return 0;
 	}
@@ -144,11 +144,10 @@
 	GLint compile_status, loglen;
 	glGetShaderiv(ret, GL_COMPILE_STATUS, &compile_status);
 	if (!compile_status) {
-		fprintf(stderr, "Shader %s failed to compile\n", fname);
 		glGetShaderiv(ret, GL_INFO_LOG_LENGTH, &loglen);
 		text = malloc(loglen);
 		glGetShaderInfoLog(ret, loglen, NULL, text);
-		fputs(text, stderr);
+		warning("Shader %s failed to compile:\n%s\n", fname, text);
 		free(text);
 		glDeleteShader(ret);
 		return 0;
@@ -259,22 +258,19 @@
 	main_context = SDL_GL_CreateContext(main_window);
 	GLenum res = glewInit();
 	if (res != GLEW_OK) {
-		fprintf(stderr, "Initialization of GLEW failed with code %d\n", res);
+		warning("Initialization of GLEW failed with code %d\n", res);
 	}
 
-	if (GLEW_VERSION_2_0) {
+	if (res == GLEW_OK && GLEW_VERSION_2_0) {
 		render_gl = 1;
-	}
-	else {
+	} else {
 		SDL_DestroyWindow(main_window);
-		fputs("OpenGL 2.0 is unavailable, falling back to SDL2 renderer\n", stderr);
+		warning("OpenGL 2.0 is unavailable, falling back to SDL2 renderer\n", stderr);
 #endif
 		SDL_CreateWindowAndRenderer(width, height, flags, &main_window, &main_renderer);
 
 		if (!main_window || !main_renderer) {
-			fprintf(stderr, "unable to create SDL window: %s\n", SDL_GetError());
-			SDL_Quit();
-			exit(1);
+			fatal_error("unable to create SDL window: %s\n", SDL_GetError());
 		}
 		main_clip.x = main_clip.y = 0;
 		main_clip.w = width;