0a8bd8
From 4b56fa058a45b7c804d1a5fcaf7a70db0bd0581c Mon Sep 17 00:00:00 2001
0a8bd8
From: =?UTF-8?q?Petr=20P=C3=ADsa=C5=99?= <petr.pisar@atlas.cz>
0a8bd8
Date: Tue, 1 Jan 2013 21:25:15 +0100
0a8bd8
Subject: [PATCH] x11: Bypass SetGammaRamp when changing gamma
0a8bd8
0a8bd8
Recent Xorg has broken dynamic colors setting, so calling SDL_SetGamme()
0a8bd8
does not have any effect here. Recent means xorg-server >= 1.7, since 2010.
0a8bd8
See <https://bugs.freedesktop.org/show_bug.cgi?id=27222>.
0a8bd8
---
0a8bd8
 src/video/SDL_gamma.c |   15 ++-------------
0a8bd8
 1 files changed, 2 insertions(+), 13 deletions(-)
0a8bd8
0a8bd8
diff --git a/src/video/SDL_gamma.c b/src/video/SDL_gamma.c
0a8bd8
index 4fd0370..464ab88 100644
0a8bd8
--- a/src/video/SDL_gamma.c
0a8bd8
+++ b/src/video/SDL_gamma.c
0a8bd8
@@ -92,22 +92,11 @@ static void CalculateGammaFromRamp(float *gamma, Uint16 *ramp)
0a8bd8
 
0a8bd8
 int SDL_SetGamma(float red, float green, float blue)
0a8bd8
 {
0a8bd8
-	int succeeded;
0a8bd8
+	int succeeded = -1;
0a8bd8
 	SDL_VideoDevice *video = current_video;
0a8bd8
 	SDL_VideoDevice *this  = current_video;	
0a8bd8
 
0a8bd8
-	succeeded = -1;
0a8bd8
-	/* Prefer using SetGammaRamp(), as it's more flexible */
0a8bd8
-	{
0a8bd8
-		Uint16 ramp[3][256];
0a8bd8
-
0a8bd8
-		CalculateGammaRamp(red, ramp[0]);
0a8bd8
-		CalculateGammaRamp(green, ramp[1]);
0a8bd8
-		CalculateGammaRamp(blue, ramp[2]);
0a8bd8
-		succeeded = SDL_SetGammaRamp(ramp[0], ramp[1], ramp[2]);
0a8bd8
-	}
0a8bd8
-	if ( (succeeded < 0) && video->SetGamma ) {
0a8bd8
-		SDL_ClearError();
0a8bd8
+	if ( video->SetGamma ) {
0a8bd8
 		succeeded = video->SetGamma(this, red, green, blue);
0a8bd8
 	}
0a8bd8
 	return succeeded;
0a8bd8
-- 
0a8bd8
1.7.8.6
0a8bd8