[Bf-blender-cvs] [82648a8] blender2.8: OpenGL: make platform requirements explicit
Mike Erwin
noreply at git.blender.org
Sat Oct 1 01:10:58 CEST 2016
Commit: 82648a8f919cf42355c3be3d1feabab0c5889508
Author: Mike Erwin
Date: Fri Sep 30 19:10:30 2016 -0400
Branches: blender2.8
https://developer.blender.org/rB82648a8f919cf42355c3be3d1feabab0c5889508
OpenGL: make platform requirements explicit
At context startup, make sure our assumptions about the OpenGL version are true. Should match since we set up the contexts... but this is what asserts are for, to check "should"s!
Part of T49012
===================================================================
M source/blender/gpu/intern/gpu_extensions.c
===================================================================
diff --git a/source/blender/gpu/intern/gpu_extensions.c b/source/blender/gpu/intern/gpu_extensions.c
index 26c863d..1e283b5 100644
--- a/source/blender/gpu/intern/gpu_extensions.c
+++ b/source/blender/gpu/intern/gpu_extensions.c
@@ -128,10 +128,21 @@ void GPU_get_dfdy_factors(float fac[2])
void gpu_extensions_init(void)
{
- BLI_assert(GLEW_VERSION_3_0 || /* Mesa mininum requirement */
- (GLEW_VERSION_2_1 && GLEW_EXT_gpu_shader4 /* Mac minimum requirement */
- && GLEW_ARB_framebuffer_object
- && GLEW_APPLE_flush_buffer_range));
+ /* during 2.8 development each platform has its own OpenGL minimum requirements
+ * final 2.8 release will be unified on OpenGL 3.2 core profile, no required extensions
+ * see developer.blender.org/T49012 for details
+ */
+#ifdef _WIN32
+ BLI_assert(GLEW_VERSION_3_2);
+#elif defined(__APPLE__)
+ BLI_assert(GLEW_VERSION_2_1 && GLEW_EXT_gpu_shader4
+ && GLEW_ARB_framebuffer_object
+ && GLEW_ARB_draw_elements_base_vertex
+ && GLEW_APPLE_flush_buffer_range);
+#else
+ BLI_assert(GLEW_VERSION_3_2 || (GLEW_VERSION_3_0 && ARB_draw_elements_base_vertex));
+ /* vendor driver || Mesa compatibility profile */
+#endif
glGetIntegerv(GL_MAX_TEXTURE_IMAGE_UNITS, &GG.maxtextures);
More information about the Bf-blender-cvs
mailing list