Horrid hack to fix loading of sRGB TIFF, for example; may break log/lin DPX again.

diff --git a/src/lib/magick_image_proxy.cc b/src/lib/magick_image_proxy.cc
index fbb08b7b081229372ddb8fd83bc32f6607971e5e..e6733c7de24552d8b807f64a79a9332be83aed4b 100644 (file)
--- a/src/lib/magick_image_proxy.cc
+++ b/src/lib/magick_image_proxy.cc
@@ -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.
                */