Make Input an abstract base class, and move the current functionality into FlatInput...

[movit] / colorspace_conversion_effect.h

diff --git a/colorspace_conversion_effect.h b/colorspace_conversion_effect.h
index 3b21804cac103f56b2d71eaecc7ae93139266b3f..e3c60ea81730af2a2eaaffe0370e23b3a71cf35c 100644 (file)
--- a/colorspace_conversion_effect.h
+++ b/colorspace_conversion_effect.h
@@ -1,6 +1,13 @@
 #ifndef _COLORSPACE_CONVERSION_EFFECT_H
 #define _COLORSPACE_CONVERSION_EFFECT_H 1
 
+// An effect to convert between different color spaces.
+// Can convert freely between sRGB/Rec. 709 and the two different Rec. 601
+// color spaces (which thankfully have the same white point).
+//
+// We don't do any fancy gamut mapping or similar; colors that are out-of-gamut
+// will simply stay out-of-gamut, and probably clip in the output stage.
+
 #include "effect.h"
 #include "effect_chain.h"
 
@@ -9,7 +16,7 @@ public:
        ColorSpaceConversionEffect();
        std::string output_fragment_shader();
 
-       virtual bool needs_srgb_primaries() { return false; }
+       virtual bool needs_srgb_primaries() const { return false; }
 
 private:
        ColorSpace source_space, destination_space;