Wrap path/data in a struct.

[dcpomatic.git] / src / lib / font.cc

diff --git a/src/lib/font.cc b/src/lib/font.cc
index f750bfc88401b4a69939ea993aecf6ed35b97a0a..f3fbc68972d0885675ae1d19c1f05773084fee10 100644 (file)
--- a/src/lib/font.cc
+++ b/src/lib/font.cc
@@ -28,6 +28,7 @@ LIBDCP_ENABLE_WARNINGS
 
 
 using std::string;
+using boost::optional;
 using namespace dcpomatic;
 
 
@@ -37,7 +38,7 @@ Font::Font (cxml::NodePtr node)
        for (auto i: node->node_children("File")) {
                string variant = i->optional_string_attribute("Variant").get_value_or("Normal");
                if (variant == "Normal") {
-                       _file = i->content();
+                       _content.file = i->content();
                }
        }
 }
@@ -47,8 +48,8 @@ void
 Font::as_xml (xmlpp::Node* node)
 {
        node->add_child("Id")->add_child_text(_id);
-       if (_file) {
-               node->add_child("File")->add_child_text(_file->string());
+       if (_content.file) {
+               node->add_child("File")->add_child_text(_content.file->string());
        }
 }
 
@@ -60,6 +61,11 @@ dcpomatic::operator== (Font const & a, Font const & b)
                return false;
        }
 
+       /* XXX: it's dubious that this ignores _data, though I think it's OK for the cases
+        * where operator== is used.  Perhaps we should remove operator== and have a more
+        * specific comparator.
+        */
+
        return a.file() == b.file();
 }
 
@@ -69,3 +75,19 @@ dcpomatic::operator!= (Font const & a, Font const & b)
 {
        return !(a == b);
 }
+
+
+optional<dcp::ArrayData>
+Font::data () const
+{
+       if (_content.data) {
+               return _content.data;
+       }
+
+       if (_content.file) {
+               return dcp::ArrayData(*_content.file);
+       }
+
+       return {};
+}
+