Add comments.

[libdcp.git] / src / reel_asset.cc

diff --git a/src/reel_asset.cc b/src/reel_asset.cc
index 62ee87bdfb573ab79907489507edfe7e3c0b32fc..037a39780e55a4a5f43eee630da1955ce77611a6 100644 (file)
--- a/src/reel_asset.cc
+++ b/src/reel_asset.cc
@@ -25,6 +25,7 @@
 
 using std::pair;
 using std::string;
+using std::stringstream;
 using std::make_pair;
 using boost::shared_ptr;
 using namespace dcp;
@@ -46,7 +47,7 @@ ReelAsset::ReelAsset ()
  *  @param intrinsic_duration Intrinsic duration of this content.
  *  @param entry_point Entry point to use in that content.
  */
-ReelAsset::ReelAsset (boost::shared_ptr<Content> content, Fraction edit_rate, int64_t intrinsic_duration, int64_t entry_point)
+ReelAsset::ReelAsset (shared_ptr<Content> content, Fraction edit_rate, int64_t intrinsic_duration, int64_t entry_point)
        : Object (content->id ())
        , _content (content)
        , _edit_rate (edit_rate)
@@ -59,7 +60,7 @@ ReelAsset::ReelAsset (boost::shared_ptr<Content> content, Fraction edit_rate, in
         _annotation_text = content->file().leaf().string ();
 }
 
-ReelAsset::ReelAsset (boost::shared_ptr<const cxml::Node> node)
+ReelAsset::ReelAsset (shared_ptr<const cxml::Node> node)
        : Object (node->string_child ("Id"))
        , _content (_id)
        , _annotation_text (node->optional_string_child ("AnnotationText").get_value_or (""))
@@ -68,22 +69,17 @@ ReelAsset::ReelAsset (boost::shared_ptr<const cxml::Node> node)
        , _entry_point (node->number_child<int64_t> ("EntryPoint"))
        , _duration (node->number_child<int64_t> ("Duration"))
        , _hash (node->optional_string_child ("Hash").get_value_or (""))
-       , _key_id (node->optional_string_child ("KeyId").get_value_or (""))
 {
        if (_id.length() > 9) {
                _id = _id.substr (9);
                _content.set_id (_id);
        }
-       
-       if (_key_id.length() > 9) {
-               _key_id = _key_id.substr (9);
-       }
 }
 
 void
-ReelAsset::write_to_cpl (xmlpp::Node* node, Standard) const
+ReelAsset::write_to_cpl (xmlpp::Node* node, Standard standard) const
 {
-        pair<string, string> const attr = cpl_node_attribute ();
+        pair<string, string> const attr = cpl_node_attribute (standard);
         xmlpp::Element* a = node->add_child (cpl_node_name ());
         if (!attr.first.empty ()) {
                 a->set_attribute (attr.first, attr.second);
@@ -94,14 +90,57 @@ ReelAsset::write_to_cpl (xmlpp::Node* node, Standard) const
         a->add_child("IntrinsicDuration")->add_child_text (raw_convert<string> (_intrinsic_duration));
         a->add_child("EntryPoint")->add_child_text (raw_convert<string> (_entry_point));
         a->add_child("Duration")->add_child_text (raw_convert<string> (_duration));
-        if (!_key_id.empty ()) {
-                a->add_child("KeyId")->add_child_text ("urn:uuid:" + _key_id);
-        }
        a->add_child("Hash")->add_child_text (_content.object()->hash ());
 }
 
 pair<string, string>
-ReelAsset::cpl_node_attribute () const
+ReelAsset::cpl_node_attribute (Standard) const
 {
        return make_pair ("", "");
 }
+
+bool
+ReelAsset::equals (shared_ptr<const ReelAsset> other, EqualityOptions opt, NoteHandler note) const
+{
+       if (_annotation_text != other->_annotation_text) {
+               stringstream s;
+               s << "Reel: annotation texts differ (" << _annotation_text << " vs " << other->_annotation_text << ")\n";
+               note (DCP_ERROR, s.str ());
+               return false;
+       }
+
+       if (_edit_rate != other->_edit_rate) {
+               note (DCP_ERROR, "Reel: edit rates differ");
+               return false;
+       }
+
+       if (_intrinsic_duration != other->_intrinsic_duration) {
+               note (DCP_ERROR, String::compose ("Reel: intrinsic durations differ (%1 vs %2)", _intrinsic_duration, other->_intrinsic_duration));
+               return false;
+       }
+
+       if (_entry_point != other->_entry_point) {
+               note (DCP_ERROR, "Reel: entry points differ");
+               return false;
+       }
+
+       if (_duration != other->_duration) {
+               note (DCP_ERROR, "Reel: durations differ");
+               return false;
+       }
+
+       if (_hash != other->_hash) {
+               if (!opt.reel_hashes_can_differ) {
+                       note (DCP_ERROR, "Reel: hashes differ");
+                       return false;
+               } else {
+                       note (DCP_NOTE, "Reel: hashes differ");
+               }
+       }
+
+       if (_content.resolved () && other->_content.resolved ()) {
+               return _content->equals (other->_content.object (), opt, note);
+       }
+
+       return true;
+}