Skip to content

Commit befe4fa

Browse files
committed
Always assume MacOS to be sRGB capable.
1 parent 407807d commit befe4fa

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

jme3-core/src/main/java/com/jme3/renderer/opengl/GLRenderer.java

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,8 @@
4848
import com.jme3.shader.*;
4949
import com.jme3.shader.Shader.ShaderSource;
5050
import com.jme3.shader.Shader.ShaderType;
51+
import com.jme3.system.JmeSystem;
52+
import com.jme3.system.Platform;
5153
import com.jme3.texture.FrameBuffer;
5254
import com.jme3.texture.FrameBuffer.RenderBuffer;
5355
import com.jme3.texture.Image;
@@ -3268,9 +3270,12 @@ public void setMainFrameBufferSrgb(boolean enableSrgb) {
32683270
setFrameBuffer(null);
32693271

32703272
if (enableSrgb) {
3271-
if (!getBoolean(GLExt.GL_FRAMEBUFFER_SRGB_CAPABLE_EXT)) {
3272-
logger.warning("Driver claims that default framebuffer "
3273-
+ "is not sRGB capable. Enabling anyway.");
3273+
if (
3274+
// Workaround: getBoolean(GLExt.GL_FRAMEBUFFER_SRGB_CAPABLE_EXT) causes error 1280 (invalid enum) on macos
3275+
JmeSystem.getPlatform().getOs() != Platform.Os.MacOS
3276+
&& !getBoolean(GLExt.GL_FRAMEBUFFER_SRGB_CAPABLE_EXT)
3277+
) {
3278+
logger.warning("Driver claims that default framebuffer " + "is not sRGB capable. Enabling anyway.");
32743279
}
32753280

32763281
gl.glEnable(GLExt.GL_FRAMEBUFFER_SRGB_EXT);

0 commit comments

Comments
 (0)