Make terminate_threads() less likely to leave _threads containing invalid pointers.
[dcpomatic.git] / src / lib / text_content.cc
index fbc7d82af9cbef12b1b9c8061c144f50dd106a5b..772ed31f56fe1ed3dfdda31a95f72a1291abe2af 100644 (file)
@@ -56,8 +56,9 @@ int const TextContentProperty::FADE_IN = 512;
 int const TextContentProperty::FADE_OUT = 513;
 int const TextContentProperty::OUTLINE_WIDTH = 514;
 int const TextContentProperty::TYPE = 515;
+int const TextContentProperty::DCP_TRACK = 516;
 
-TextContent::TextContent (Content* parent, TextType original_type)
+TextContent::TextContent (Content* parent, TextType type, TextType original_type)
        : ContentPart (parent)
        , _use (false)
        , _burn (false)
@@ -66,14 +67,14 @@ TextContent::TextContent (Content* parent, TextType original_type)
        , _x_scale (1)
        , _y_scale (1)
        , _line_spacing (1)
-       , _outline_width (2)
-       , _type (original_type)
+       , _outline_width (4)
+       , _type (type)
        , _original_type (original_type)
 {
 
 }
 
-/** @return TextContents from node or <Caption> nodes under node (according to version).
+/** @return TextContents from node or <Text> nodes under node (according to version).
  *  The list could be empty if no TextContents are found.
  */
 list<shared_ptr<TextContent> >
@@ -99,14 +100,15 @@ TextContent::from_xml (Content* parent, cxml::ConstNodePtr node, int version)
                return c;
        }
 
-       if (!node->optional_node_child("Caption")) {
+       if (!node->optional_node_child("Text")) {
                return list<shared_ptr<TextContent> >();
        }
 
        list<shared_ptr<TextContent> > c;
-       BOOST_FOREACH (cxml::ConstNodePtr i, node->node_children("Caption")) {
+       BOOST_FOREACH (cxml::ConstNodePtr i, node->node_children("Text")) {
                c.push_back (shared_ptr<TextContent> (new TextContent (parent, i, version)));
        }
+
        return c;
 }
 
@@ -119,9 +121,9 @@ TextContent::TextContent (Content* parent, cxml::ConstNodePtr node, int version)
        , _x_scale (1)
        , _y_scale (1)
        , _line_spacing (node->optional_number_child<double>("LineSpacing").get_value_or (1))
-       , _outline_width (node->optional_number_child<int>("OutlineWidth").get_value_or (2))
-       , _type (CAPTION_OPEN)
-       , _original_type (CAPTION_OPEN)
+       , _outline_width (node->optional_number_child<int>("OutlineWidth").get_value_or (4))
+       , _type (TEXT_OPEN_SUBTITLE)
+       , _original_type (TEXT_OPEN_SUBTITLE)
 {
        if (version >= 37) {
                _use = node->bool_child ("Use");
@@ -225,63 +227,76 @@ TextContent::TextContent (Content* parent, cxml::ConstNodePtr node, int version)
 
        connect_to_fonts ();
 
-       _type = string_to_caption_type (node->optional_string_child("Type").get_value_or("open"));
-       _original_type = string_to_caption_type (node->optional_string_child("OriginalType").get_value_or("open"));
+       if (version >= 37) {
+               _type = string_to_text_type (node->optional_string_child("Type").get_value_or("open"));
+               if (node->optional_string_child("OriginalType")) {
+                       _original_type = string_to_text_type (node->optional_string_child("OriginalType").get());
+               }
+       }
+
+       cxml::ConstNodePtr dt = node->optional_node_child("DCPTrack");
+       if (dt) {
+               _dcp_track = DCPTextTrack (dt);
+       }
 }
 
 TextContent::TextContent (Content* parent, vector<shared_ptr<Content> > c)
        : ContentPart (parent)
 {
        /* This constructor is for join which is only supported for content types
-          that have a single caption, so we can use only_caption() here.
+          that have a single text, so we can use only_text() here.
        */
-       shared_ptr<TextContent> ref = c[0]->only_caption();
+       shared_ptr<TextContent> ref = c[0]->only_text();
        DCPOMATIC_ASSERT (ref);
        list<shared_ptr<Font> > ref_fonts = ref->fonts ();
 
        for (size_t i = 1; i < c.size(); ++i) {
 
-               if (c[i]->only_caption()->use() != ref->use()) {
+               if (c[i]->only_text()->use() != ref->use()) {
                        throw JoinError (_("Content to be joined must have the same 'use subtitles' setting."));
                }
 
-               if (c[i]->only_caption()->burn() != ref->burn()) {
+               if (c[i]->only_text()->burn() != ref->burn()) {
                        throw JoinError (_("Content to be joined must have the same 'burn subtitles' setting."));
                }
 
-               if (c[i]->only_caption()->x_offset() != ref->x_offset()) {
+               if (c[i]->only_text()->x_offset() != ref->x_offset()) {
                        throw JoinError (_("Content to be joined must have the same subtitle X offset."));
                }
 
-               if (c[i]->only_caption()->y_offset() != ref->y_offset()) {
+               if (c[i]->only_text()->y_offset() != ref->y_offset()) {
                        throw JoinError (_("Content to be joined must have the same subtitle Y offset."));
                }
 
-               if (c[i]->only_caption()->x_scale() != ref->x_scale()) {
+               if (c[i]->only_text()->x_scale() != ref->x_scale()) {
                        throw JoinError (_("Content to be joined must have the same subtitle X scale."));
                }
 
-               if (c[i]->only_caption()->y_scale() != ref->y_scale()) {
+               if (c[i]->only_text()->y_scale() != ref->y_scale()) {
                        throw JoinError (_("Content to be joined must have the same subtitle Y scale."));
                }
 
-               if (c[i]->only_caption()->line_spacing() != ref->line_spacing()) {
+               if (c[i]->only_text()->line_spacing() != ref->line_spacing()) {
                        throw JoinError (_("Content to be joined must have the same subtitle line spacing."));
                }
 
-               if ((c[i]->only_caption()->fade_in() != ref->fade_in()) || (c[i]->only_caption()->fade_out() != ref->fade_out())) {
+               if ((c[i]->only_text()->fade_in() != ref->fade_in()) || (c[i]->only_text()->fade_out() != ref->fade_out())) {
                        throw JoinError (_("Content to be joined must have the same subtitle fades."));
                }
 
-               if ((c[i]->only_caption()->outline_width() != ref->outline_width())) {
+               if ((c[i]->only_text()->outline_width() != ref->outline_width())) {
                        throw JoinError (_("Content to be joined must have the same outline width."));
                }
 
-               list<shared_ptr<Font> > fonts = c[i]->only_caption()->fonts ();
+               list<shared_ptr<Font> > fonts = c[i]->only_text()->fonts ();
                if (fonts.size() != ref_fonts.size()) {
                        throw JoinError (_("Content to be joined must use the same fonts."));
                }
 
+               if (c[i]->only_text()->dcp_track() != ref->dcp_track()) {
+                       throw JoinError (_("Content to be joined must use the same DCP track."));
+               }
+
                list<shared_ptr<Font> >::const_iterator j = ref_fonts.begin ();
                list<shared_ptr<Font> >::const_iterator k = fonts.begin ();
 
@@ -308,6 +323,7 @@ TextContent::TextContent (Content* parent, vector<shared_ptr<Content> > c)
        _outline_width = ref->outline_width ();
        _type = ref->type ();
        _original_type = ref->original_type ();
+       _dcp_track = ref->dcp_track ();
 
        connect_to_fonts ();
 }
@@ -318,53 +334,56 @@ TextContent::as_xml (xmlpp::Node* root) const
 {
        boost::mutex::scoped_lock lm (_mutex);
 
-       xmlpp::Element* caption = root->add_child ("Caption");
+       xmlpp::Element* text = root->add_child ("Text");
 
-       caption->add_child("Use")->add_child_text (_use ? "1" : "0");
-       caption->add_child("Burn")->add_child_text (_burn ? "1" : "0");
-       caption->add_child("XOffset")->add_child_text (raw_convert<string> (_x_offset));
-       caption->add_child("YOffset")->add_child_text (raw_convert<string> (_y_offset));
-       caption->add_child("XScale")->add_child_text (raw_convert<string> (_x_scale));
-       caption->add_child("YScale")->add_child_text (raw_convert<string> (_y_scale));
-       caption->add_child("Language")->add_child_text (_language);
+       text->add_child("Use")->add_child_text (_use ? "1" : "0");
+       text->add_child("Burn")->add_child_text (_burn ? "1" : "0");
+       text->add_child("XOffset")->add_child_text (raw_convert<string> (_x_offset));
+       text->add_child("YOffset")->add_child_text (raw_convert<string> (_y_offset));
+       text->add_child("XScale")->add_child_text (raw_convert<string> (_x_scale));
+       text->add_child("YScale")->add_child_text (raw_convert<string> (_y_scale));
+       text->add_child("Language")->add_child_text (_language);
        if (_colour) {
-               caption->add_child("Red")->add_child_text (raw_convert<string> (_colour->r));
-               caption->add_child("Green")->add_child_text (raw_convert<string> (_colour->g));
-               caption->add_child("Blue")->add_child_text (raw_convert<string> (_colour->b));
+               text->add_child("Red")->add_child_text (raw_convert<string> (_colour->r));
+               text->add_child("Green")->add_child_text (raw_convert<string> (_colour->g));
+               text->add_child("Blue")->add_child_text (raw_convert<string> (_colour->b));
        }
        if (_effect) {
                switch (*_effect) {
                case dcp::NONE:
-                       caption->add_child("Effect")->add_child_text("none");
+                       text->add_child("Effect")->add_child_text("none");
                        break;
                case dcp::BORDER:
-                       caption->add_child("Effect")->add_child_text("outline");
+                       text->add_child("Effect")->add_child_text("outline");
                        break;
                case dcp::SHADOW:
-                       caption->add_child("Effect")->add_child_text("shadow");
+                       text->add_child("Effect")->add_child_text("shadow");
                        break;
                }
        }
        if (_effect_colour) {
-               caption->add_child("EffectRed")->add_child_text (raw_convert<string> (_effect_colour->r));
-               caption->add_child("EffectGreen")->add_child_text (raw_convert<string> (_effect_colour->g));
-               caption->add_child("EffectBlue")->add_child_text (raw_convert<string> (_effect_colour->b));
+               text->add_child("EffectRed")->add_child_text (raw_convert<string> (_effect_colour->r));
+               text->add_child("EffectGreen")->add_child_text (raw_convert<string> (_effect_colour->g));
+               text->add_child("EffectBlue")->add_child_text (raw_convert<string> (_effect_colour->b));
        }
-       caption->add_child("LineSpacing")->add_child_text (raw_convert<string> (_line_spacing));
+       text->add_child("LineSpacing")->add_child_text (raw_convert<string> (_line_spacing));
        if (_fade_in) {
-               caption->add_child("FadeIn")->add_child_text (raw_convert<string> (_fade_in->get()));
+               text->add_child("FadeIn")->add_child_text (raw_convert<string> (_fade_in->get()));
        }
        if (_fade_out) {
-               caption->add_child("FadeOut")->add_child_text (raw_convert<string> (_fade_out->get()));
+               text->add_child("FadeOut")->add_child_text (raw_convert<string> (_fade_out->get()));
        }
-       caption->add_child("OutlineWidth")->add_child_text (raw_convert<string> (_outline_width));
+       text->add_child("OutlineWidth")->add_child_text (raw_convert<string> (_outline_width));
 
        for (list<shared_ptr<Font> >::const_iterator i = _fonts.begin(); i != _fonts.end(); ++i) {
-               (*i)->as_xml (caption->add_child("Font"));
+               (*i)->as_xml (text->add_child("Font"));
        }
 
-       caption->add_child("Type")->add_child_text (caption_type_to_string(_type));
-       caption->add_child("OriginalType")->add_child_text (caption_type_to_string(_original_type));
+       text->add_child("Type")->add_child_text (text_type_to_string(_type));
+       text->add_child("OriginalType")->add_child_text (text_type_to_string(_original_type));
+       if (_dcp_track) {
+               _dcp_track->as_xml(text->add_child("DCPTrack"));
+       }
 }
 
 string
@@ -380,18 +399,17 @@ TextContent::identifier () const
                + "_" + 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());
+               + "_" + raw_convert<string> (effect_colour().get_value_or(dcp::Colour(0, 0, 0)).to_argb_string())
+               + "_" + raw_convert<string> (_parent->video_frame_rate().get_value_or(0));
 
        /* XXX: I suppose really _fonts shouldn't be in here, since not all
           types of subtitle content involve fonts.
        */
        BOOST_FOREACH (shared_ptr<Font> f, _fonts) {
-               for (int i = 0; i < FontFiles::VARIANTS; ++i) {
-                       s += "_" + f->file(static_cast<FontFiles::Variant>(i)).get_value_or("Default").string();
-               }
+               s += "_" + f->file().get_value_or("Default").string();
        }
 
-       /* The language is for metadata only, and doesn't affect
+       /* The DCP track and language are for metadata only, and don't affect
           how this content looks.
        */
 
@@ -422,7 +440,8 @@ TextContent::connect_to_fonts ()
 void
 TextContent::font_changed ()
 {
-       _parent->signal_changed (TextContentProperty::FONTS);
+       /* XXX: too late */
+       ChangeSignaller<Content> cc (_parent, TextContentProperty::FONTS);
 }
 
 void
@@ -545,6 +564,18 @@ TextContent::set_outline_width (int w)
        maybe_set (_outline_width, w, TextContentProperty::OUTLINE_WIDTH);
 }
 
+void
+TextContent::set_dcp_track (DCPTextTrack t)
+{
+       maybe_set (_dcp_track, t, TextContentProperty::DCP_TRACK);
+}
+
+void
+TextContent::unset_dcp_track ()
+{
+       maybe_set (_dcp_track, optional<DCPTextTrack>(), TextContentProperty::DCP_TRACK);
+}
+
 void
 TextContent::take_settings_from (shared_ptr<const TextContent> c)
 {
@@ -576,4 +607,9 @@ TextContent::take_settings_from (shared_ptr<const TextContent> c)
                set_fade_out (*c->_fade_out);
        }
        set_outline_width (c->_outline_width);
+       if (c->_dcp_track) {
+               set_dcp_track (c->_dcp_track.get());
+       } else {
+               unset_dcp_track ();
+       }
 }