Fix various bugs in subtitle/ccap verification.
[libdcp.git] / src / subtitle_asset_internal.cc
index 7b9e18e90e7aa11b11f296a2e0ef8e11ebc23c59..80e861f117a9b55cae26299138197a8276bf038e 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2012-2018 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-2021 Carl Hetherington <cth@carlh.net>
 
     This file is part of libdcp.
 
     files in the program, then also delete it here.
 */
 
+
+/** @file  src/subtitle_asset_internal.cc
+ *  @brief Internal SubtitleAsset helpers
+ */
+
+
 #include "subtitle_asset_internal.h"
 #include "subtitle_string.h"
 #include "compose.hpp"
 #include <cmath>
 
+
 using std::string;
 using std::map;
-using boost::shared_ptr;
+using std::shared_ptr;
 using namespace dcp;
 
+
 string
 order::Context::xmlns () const
 {
-       return standard == SMPTE ? "dcst" : "";
+       return standard == Standard::SMPTE ? "dcst" : "";
 }
 
+
 order::Font::Font (shared_ptr<SubtitleString> s, Standard standard)
 {
        if (s->font()) {
-               if (standard == SMPTE) {
+               if (standard == Standard::SMPTE) {
                        _values["ID"] = s->font().get ();
                } else {
                        _values["Id"] = s->font().get ();
@@ -63,7 +72,7 @@ order::Font::Font (shared_ptr<SubtitleString> s, Standard standard)
        _values["Effect"] = effect_to_string (s->effect());
        _values["EffectColor"] = s->effect_colour().to_argb_string();
        _values["Script"] = "normal";
-       if (standard == SMPTE) {
+       if (standard == Standard::SMPTE) {
                _values["Underline"] = s->underline() ? "yes" : "no";
        } else {
                _values["Underlined"] = s->underline() ? "yes" : "no";
@@ -71,6 +80,7 @@ order::Font::Font (shared_ptr<SubtitleString> s, Standard standard)
        _values["Weight"] = s->bold() ? "bold" : "normal";
 }
 
+
 xmlpp::Element*
 order::Font::as_xml (xmlpp::Element* parent, Context& context) const
 {
@@ -81,6 +91,7 @@ order::Font::as_xml (xmlpp::Element* parent, Context& context) const
        return e;
 }
 
+
 /** Modify our values so that they contain only those that are common to us and
  *  other.
  */
@@ -89,42 +100,46 @@ order::Font::take_intersection (Font other)
 {
        map<string, string> inter;
 
-       for (map<string, string>::const_iterator i = other._values.begin(); i != other._values.end(); ++i) {
-               map<string, string>::iterator t = _values.find (i->first);
-               if (t != _values.end() && t->second == i->second) {
-                       inter.insert (*i);
+       for (auto const& i: other._values) {
+               auto t = _values.find (i.first);
+               if (t != _values.end() && t->second == i.second) {
+                       inter.insert (i);
                }
        }
 
        _values = inter;
 }
 
+
 /** Modify our values so that it contains only those keys that are not in other */
 void
 order::Font::take_difference (Font other)
 {
        map<string, string> diff;
-       for (map<string, string>::const_iterator i = _values.begin(); i != _values.end(); ++i) {
-               if (other._values.find (i->first) == other._values.end ()) {
-                       diff.insert (*i);
+       for (auto const& i: _values) {
+               if (other._values.find (i.first) == other._values.end()) {
+                       diff.insert (i);
                }
        }
 
        _values = diff;
 }
 
+
 bool
 order::Font::empty () const
 {
        return _values.empty ();
 }
 
+
 xmlpp::Element*
 order::Part::as_xml (xmlpp::Element* parent, Context &) const
 {
        return parent;
 }
 
+
 xmlpp::Element*
 order::String::as_xml (xmlpp::Element* parent, Context &) const
 {
@@ -132,6 +147,7 @@ order::String::as_xml (xmlpp::Element* parent, Context &) const
        return 0;
 }
 
+
 void
 order::Part::write_xml (xmlpp::Element* parent, order::Context& context) const
 {
@@ -141,16 +157,17 @@ order::Part::write_xml (xmlpp::Element* parent, order::Context& context) const
 
        parent = as_xml (parent, context);
 
-       BOOST_FOREACH (boost::shared_ptr<order::Part> i, children) {
+       for (auto i: children) {
                i->write_xml (parent, context);
        }
 }
 
+
 static void
 position_align (xmlpp::Element* e, order::Context& context, HAlign h_align, float h_position, VAlign v_align, float v_position)
 {
-       if (h_align != HALIGN_CENTER) {
-               if (context.standard == SMPTE) {
+       if (h_align != HAlign::CENTER) {
+               if (context.standard == Standard::SMPTE) {
                        e->set_attribute ("Halign", halign_to_string (h_align));
                } else {
                        e->set_attribute ("HAlign", halign_to_string (h_align));
@@ -158,27 +175,27 @@ position_align (xmlpp::Element* e, order::Context& context, HAlign h_align, floa
        }
 
        if (fabs(h_position) > ALIGN_EPSILON) {
-               if (context.standard == SMPTE) {
+               if (context.standard == Standard::SMPTE) {
                        e->set_attribute ("Hposition", raw_convert<string> (h_position * 100, 6));
                } else {
                        e->set_attribute ("HPosition", raw_convert<string> (h_position * 100, 6));
                }
        }
 
-       if (context.standard == SMPTE) {
+       if (context.standard == Standard::SMPTE) {
                e->set_attribute ("Valign", valign_to_string (v_align));
        } else {
                e->set_attribute ("VAlign", valign_to_string (v_align));
        }
 
        if (fabs(v_position) > ALIGN_EPSILON) {
-               if (context.standard == SMPTE) {
+               if (context.standard == Standard::SMPTE) {
                        e->set_attribute ("Vposition", raw_convert<string> (v_position * 100, 6));
                } else {
                        e->set_attribute ("VPosition", raw_convert<string> (v_position * 100, 6));
                }
        } else {
-               if (context.standard == SMPTE) {
+               if (context.standard == Standard::SMPTE) {
                        e->set_attribute ("Vposition", "0");
                } else {
                        e->set_attribute ("VPosition", "0");
@@ -186,59 +203,64 @@ position_align (xmlpp::Element* e, order::Context& context, HAlign h_align, floa
        }
 }
 
+
 xmlpp::Element*
 order::Text::as_xml (xmlpp::Element* parent, Context& context) const
 {
-       xmlpp::Element* e = parent->add_child ("Text", context.xmlns());
+       auto e = parent->add_child ("Text", context.xmlns());
 
        position_align (e, context, _h_align, _h_position, _v_align, _v_position);
 
        /* Interop only supports "horizontal" or "vertical" for direction, so only write this
           for SMPTE.
        */
-       if (_direction != DIRECTION_LTR && context.standard == SMPTE) {
+       if (_direction != Direction::LTR && context.standard == Standard::SMPTE) {
                e->set_attribute ("Direction", direction_to_string (_direction));
        }
 
        return e;
 }
 
+
 xmlpp::Element*
 order::Subtitle::as_xml (xmlpp::Element* parent, Context& context) const
 {
-       xmlpp::Element* e = parent->add_child ("Subtitle", context.xmlns());
+       auto e = parent->add_child ("Subtitle", context.xmlns());
        e->set_attribute ("SpotNumber", raw_convert<string> (context.spot_number++));
        e->set_attribute ("TimeIn", _in.rebase(context.time_code_rate).as_string(context.standard));
        e->set_attribute ("TimeOut", _out.rebase(context.time_code_rate).as_string(context.standard));
-       if (context.standard == SMPTE) {
+       if (context.standard == Standard::SMPTE) {
                e->set_attribute ("FadeUpTime", _fade_up.rebase(context.time_code_rate).as_string(context.standard));
                e->set_attribute ("FadeDownTime", _fade_down.rebase(context.time_code_rate).as_string(context.standard));
        } else {
-               e->set_attribute ("FadeUpTime", raw_convert<string> (_fade_up.as_editable_units(context.time_code_rate)));
-               e->set_attribute ("FadeDownTime", raw_convert<string> (_fade_down.as_editable_units(context.time_code_rate)));
+               e->set_attribute ("FadeUpTime", raw_convert<string> (_fade_up.as_editable_units_ceil(context.time_code_rate)));
+               e->set_attribute ("FadeDownTime", raw_convert<string> (_fade_down.as_editable_units_ceil(context.time_code_rate)));
        }
        return e;
 }
 
+
 bool
 order::Font::operator== (Font const & other) const
 {
        return _values == other._values;
 }
 
+
 void
 order::Font::clear ()
 {
        _values.clear ();
 }
 
+
 xmlpp::Element *
 order::Image::as_xml (xmlpp::Element* parent, Context& context) const
 {
-       xmlpp::Element* e = parent->add_child ("Image", context.xmlns());
+       auto e = parent->add_child ("Image", context.xmlns());
 
        position_align (e, context, _h_align, _h_position, _v_align, _v_position);
-       if (context.standard == SMPTE) {
+       if (context.standard == Standard::SMPTE) {
                e->add_child_text (_id);
        } else {
                e->add_child_text (_id + ".png");