X-Git-Url: https://git.sesse.net/?p=movit;a=blobdiff_plain;f=effect_chain.cpp;h=312b27da9eef4a9ee59f09f4f5752a69e4464a1f;hp=82e9c5d705e09b2b1ff5552baf13f1996b8e4e50;hb=ab636361e865c389fe60584562372ec39ac4c511;hpb=0dc1dfe6444a700ebd2c9f006cba000b90c3a7b0

diff --git a/effect_chain.cpp b/effect_chain.cpp
index 82e9c5d..312b27d 100644
--- a/effect_chain.cpp
+++ b/effect_chain.cpp
@@ -4,9 +4,6 @@
 #include <string.h>
 #include <assert.h>
 
-#include <GL/gl.h>
-#include <GL/glext.h>
-
 #include <algorithm>
 #include <set>
 #include <stack>
@@ -18,6 +15,7 @@
 #include "gamma_compression_effect.h"
 #include "colorspace_conversion_effect.h"
 #include "input.h"
+#include "opengl.h"
 
 EffectChain::EffectChain(unsigned width, unsigned height)
 	: width(width),
@@ -60,48 +58,70 @@ void EffectChain::add_effect_raw(Effect *effect, const std::vector<Effect *> &in
 	output_color_space[effect] = output_color_space[last_added_effect()];
 }
 
-// Set the "use_srgb_texture_format" option on all inputs that feed into this node,
-// and update the output_gamma_curve[] map as we go.
-//
-// NOTE: We assume that the only way we could actually get GAMMA_sRGB from an
-// effect (except from GammaCompressionCurve, which should never be inserted
-// into a chain when this is called) is by pass-through from a texture.
-// Thus, we can simply feed the flag up towards all inputs.
-void EffectChain::set_use_srgb_texture_format(Effect *effect)
+void EffectChain::find_all_nonlinear_inputs(Effect *effect,
+                                            std::vector<Input *> *nonlinear_inputs,
+                                            std::vector<Effect *> *intermediates)
 {
 	assert(output_gamma_curve.count(effect) != 0);
-	assert(output_gamma_curve[effect] == GAMMA_sRGB);
+	if (output_gamma_curve[effect] == GAMMA_LINEAR) {
+		return;
+	}
 	if (effect->num_inputs() == 0) {
-		effect->set_int("use_srgb_texture_format", 1);
+		nonlinear_inputs->push_back(static_cast<Input *>(effect));
 	} else {
+		intermediates->push_back(effect);
+
 		assert(incoming_links.count(effect) == 1);
 		std::vector<Effect *> deps = incoming_links[effect];
 		assert(effect->num_inputs() == deps.size());
 		for (unsigned i = 0; i < deps.size(); ++i) {
-			set_use_srgb_texture_format(deps[i]);
-			assert(output_gamma_curve[deps[i]] == GAMMA_LINEAR);
+			find_all_nonlinear_inputs(deps[i], nonlinear_inputs, intermediates);
 		}
 	}
-	output_gamma_curve[effect] = GAMMA_LINEAR;
 }
 
 Effect *EffectChain::normalize_to_linear_gamma(Effect *input)
 {
-	assert(output_gamma_curve.count(input) != 0);
-	if (output_gamma_curve[input] == GAMMA_sRGB) {
-		// TODO: check if the extension exists
-		set_use_srgb_texture_format(input);
-		output_gamma_curve[input] = GAMMA_LINEAR;
+	// Find out if all the inputs can be set to deliver sRGB inputs.
+	// If so, we can just ask them to do that instead of inserting a
+	// (possibly expensive) conversion operation.
+	//
+	// NOTE: We assume that effects generally don't mess with the gamma
+	// curve (except GammaCompressionEffect, which should never be
+	// inserted into a chain when this is called), so that we can just
+	// update the output gamma as we go.
+	//
+	// TODO: Setting this flag for one source might confuse a different
+	// part of the pipeline using the same source.
+	std::vector<Input *> nonlinear_inputs;
+	std::vector<Effect *> intermediates;
+	find_all_nonlinear_inputs(input, &nonlinear_inputs, &intermediates);
+
+	bool all_ok = true;
+	for (unsigned i = 0; i < nonlinear_inputs.size(); ++i) {
+		all_ok &= nonlinear_inputs[i]->can_output_linear_gamma();
+	}
+
+	if (all_ok) {
+		for (unsigned i = 0; i < nonlinear_inputs.size(); ++i) {
+			bool ok = nonlinear_inputs[i]->set_int("output_linear_gamma", 1);
+			assert(ok);
+			output_gamma_curve[nonlinear_inputs[i]] = GAMMA_LINEAR;
+		}
+		for (unsigned i = 0; i < intermediates.size(); ++i) {
+			output_gamma_curve[intermediates[i]] = GAMMA_LINEAR;
+		}
 		return input;
-	} else {
-		GammaExpansionEffect *gamma_conversion = new GammaExpansionEffect();
-		gamma_conversion->set_int("source_curve", output_gamma_curve[input]);
-		std::vector<Effect *> inputs;
-		inputs.push_back(input);
-		gamma_conversion->add_self_to_effect_chain(this, inputs);
-		output_gamma_curve[gamma_conversion] = GAMMA_LINEAR;
-		return gamma_conversion;
 	}
+
+	// OK, that didn't work. Insert a conversion effect.
+	GammaExpansionEffect *gamma_conversion = new GammaExpansionEffect();
+	gamma_conversion->set_int("source_curve", output_gamma_curve[input]);
+	std::vector<Effect *> inputs;
+	inputs.push_back(input);
+	gamma_conversion->add_self_to_effect_chain(this, inputs);
+	output_gamma_curve[gamma_conversion] = GAMMA_LINEAR;
+	return gamma_conversion;
 }
 
 Effect *EffectChain::normalize_to_srgb(Effect *input)