Add colour, effect and effect colour to ::identifier() to force remake of video when...

diff --git a/ChangeLog b/ChangeLog
index bbbee5ff9786c7590c9bbc4d95d4808ffac20177..e397d795226ff6cb4753a2cb3768e254ca78689a 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2018-05-08  Carl Hetherington  <cth@carlh.net>
+
+       * Look at subtitle colour and effect when deciding whether or not to
+       re-make video (should fix #1290).
+
 2018-05-07  Carl Hetherington  <cth@carlh.net>
 
        * Updated cs_CZ translation from Tomáš Begeni.

diff --git a/src/lib/subtitle_content.cc b/src/lib/subtitle_content.cc
index 9e3ed322969b71b3e65fedec84d5f0824b510b41..cd93638398ce4777cd511388bfdb53e62c4db03b 100644 (file)
--- a/src/lib/subtitle_content.cc
+++ b/src/lib/subtitle_content.cc
@@ -319,7 +319,10 @@ SubtitleContent::identifier () const
                + "_" + raw_convert<string> (line_spacing())
                + "_" + raw_convert<string> (fade_in().get_value_or(ContentTime()).get())
                + "_" + raw_convert<string> (fade_out().get_value_or(ContentTime()).get())
-               + "_" + raw_convert<string> (outline_width());
+               + "_" + raw_convert<string> (outline_width())
+               + "_" + raw_convert<string> (colour().get_value_or(dcp::Colour(255, 255, 255)).to_argb_string())
+               + "_" + raw_convert<string> (dcp::effect_to_string(effect().get_value_or(dcp::NONE)))
+               + "_" + raw_convert<string> (effect_colour().get_value_or(dcp::Colour(0, 0, 0)).to_argb_string());
 
        /* XXX: I suppose really _fonts shouldn't be in here, since not all
           types of subtitle content involve fonts.