, _x_scale (1)
, _y_scale (1)
, _line_spacing (node->optional_number_child<double>("LineSpacing").get_value_or (1))
- , _fade_in (node->optional_number_child<Frame>("SubtitleFadeIn").get_value_or (0))
- , _fade_out (node->optional_number_child<Frame>("SubtitleFadeOut").get_value_or (0))
, _outline_width (node->optional_number_child<int>("OutlineWidth").get_value_or (2))
{
if (version >= 32) {
);
}
+ optional<Frame> fi = node->optional_number_child<Frame>("SubtitleFadeIn");
+ if (fi) {
+ _fade_in = ContentTime (*fi);
+ }
+ optional<Frame> fo = node->optional_number_child<Frame>("SubtitleFadeOut");
+ if (fo) {
+ _fade_out = ContentTime (*fo);
+ }
+
_language = node->optional_string_child ("SubtitleLanguage").get_value_or ("");
list<cxml::NodePtr> fonts = node->node_children ("Font");
if (_effect) {
switch (*_effect) {
case dcp::NONE:
- root->add_child("none");
+ root->add_child("Effect")->add_child_text("none");
break;
case dcp::BORDER:
- root->add_child("outline");
+ root->add_child("Effect")->add_child_text("outline");
break;
case dcp::SHADOW:
- root->add_child("shadow");
+ root->add_child("Effect")->add_child_text("shadow");
break;
}
}
root->add_child("EffectBlue")->add_child_text (raw_convert<string> (_effect_colour->b));
}
root->add_child("LineSpacing")->add_child_text (raw_convert<string> (_line_spacing));
- root->add_child("SubtitleFadeIn")->add_child_text (raw_convert<string> (_fade_in.get()));
- root->add_child("SubtitleFadeOut")->add_child_text (raw_convert<string> (_fade_out.get()));
+ if (_fade_in) {
+ root->add_child("SubtitleFadeIn")->add_child_text (raw_convert<string> (_fade_in->get()));
+ }
+ if (_fade_out) {
+ root->add_child("SubtitleFadeOut")->add_child_text (raw_convert<string> (_fade_out->get()));
+ }
root->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) {
+ "_" + raw_convert<string> (x_offset())
+ "_" + raw_convert<string> (y_offset())
+ "_" + raw_convert<string> (line_spacing())
- + "_" + raw_convert<string> (fade_in().get())
- + "_" + raw_convert<string> (fade_out().get())
+ + "_" + 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());
/* XXX: I suppose really _fonts shouldn't be in here, since not all
maybe_set (_fade_in, t, SubtitleContentProperty::FADE_IN);
}
+void
+SubtitleContent::unset_fade_in ()
+{
+ maybe_set (_fade_in, optional<ContentTime>(), SubtitleContentProperty::FADE_IN);
+}
+
void
SubtitleContent::set_fade_out (ContentTime t)
{
maybe_set (_fade_out, t, SubtitleContentProperty::FADE_OUT);
}
+void
+SubtitleContent::unset_fade_out ()
+{
+ maybe_set (_fade_out, optional<ContentTime>(), SubtitleContentProperty::FADE_OUT);
+}
+
void
SubtitleContent::set_outline_width (int w)
{
unset_effect_colour ();
}
set_line_spacing (c->_line_spacing);
- set_fade_in (c->_fade_in);
- set_fade_out (c->_fade_out);
+ if (c->_fade_in) {
+ set_fade_in (*c->_fade_in);
+ }
+ if (c->_fade_out) {
+ set_fade_out (*c->_fade_out);
+ }
set_outline_width (c->_outline_width);
}