Horrid hack to fix loading of sRGB TIFF, for example; may break log/lin DPX again.
authorCarl Hetherington <cth@carlh.net>
Tue, 10 Oct 2017 22:05:54 +0000 (23:05 +0100)
committerCarl Hetherington <cth@carlh.net>
Tue, 10 Oct 2017 22:05:54 +0000 (23:05 +0100)
src/lib/magick_image_proxy.cc

index fbb08b7..e6733c7 100644 (file)
@@ -103,9 +103,13 @@ MagickImageProxy::image (optional<dcp::NoteHandler>, optional<dcp::Size>) const
        }
 
        unsigned char const * data = static_cast<unsigned char const *>(_blob.data());
-       if (data[801] == 1) {
-               /* The transfer characteristic in this file is "printing density"; in this case ImageMagick sets the colour space
-                  to LogColorspace.  Empirically we find that if we subsequently call colorSpace(Magick::RGBColorspace) the colours
+       if (data[801] == 1 || magick_image->image()->colorspace == Magick::sRGBColorspace) {
+               /* Either:
+                  1.  The transfer characteristic in this file is "printing density"; in this case ImageMagick sets the colour space
+                      to LogColorspace, or
+                  2.  The file is sRGB.
+
+                  Empirically we find that in these cases if we subsequently call colorSpace(Magick::RGBColorspace) the colours
                   are very wrong.  To prevent this, set the image colour space to RGB to stop the ::colorSpace call below doing
                   anything.  See #1123 and others.
                */