Supporters update.
[dcpomatic.git] / src / lib / dcp_content.cc
index 928ba33818a65e6b6b8f141632b54f2d6121e521..bdd5e0e091c0aa6b523ec7c7400ec9449beef3a0 100644 (file)
@@ -31,7 +31,6 @@
 #include "job.h"
 #include "log.h"
 #include "overlaps.h"
-#include "scope_guard.h"
 #include "text_content.h"
 #include "video_content.h"
 #include <dcp/dcp.h>
@@ -41,6 +40,7 @@
 #include <dcp/reel_picture_asset.h>
 #include <dcp/reel_subtitle_asset.h>
 #include <dcp/reel.h>
+#include <dcp/scope_guard.h>
 #include <libxml++/libxml++.h>
 #include <iterator>
 #include <iostream>
@@ -224,7 +224,7 @@ DCPContent::examine (shared_ptr<const Film> film, shared_ptr<Job> job)
        string const old_name = name ();
 
        ContentChangeSignalDespatcher::instance()->suspend();
-       ScopeGuard sg = []() {
+       dcp::ScopeGuard sg = []() {
                ContentChangeSignalDespatcher::instance()->resume();
        };
 
@@ -280,14 +280,14 @@ DCPContent::examine (shared_ptr<const Film> film, shared_ptr<Job> job)
        for (int i = 0; i < examiner->text_count(TextType::OPEN_SUBTITLE); ++i) {
                auto c = make_shared<TextContent>(this, TextType::OPEN_SUBTITLE, TextType::OPEN_SUBTITLE);
                c->set_language (examiner->open_subtitle_language());
-               add_fonts_from_examiner(c, examiner->fonts());
+               examiner->add_fonts(c);
                new_text.push_back (c);
        }
 
        for (int i = 0; i < examiner->text_count(TextType::CLOSED_CAPTION); ++i) {
                auto c = make_shared<TextContent>(this, TextType::CLOSED_CAPTION, TextType::CLOSED_CAPTION);
                c->set_dcp_track (examiner->dcp_text_track(i));
-               add_fonts_from_examiner(c, examiner->fonts());
+               examiner->add_fonts(c);
                new_text.push_back (c);
        }
 
@@ -374,6 +374,7 @@ DCPContent::as_xml (xmlpp::Node* node, bool with_paths) const
        }
 
        boost::mutex::scoped_lock lm (_mutex);
+
        node->add_child("Name")->add_child_text (_name);
        node->add_child("Encrypted")->add_child_text (_encrypted ? "1" : "0");
        node->add_child("NeedsAssets")->add_child_text (_needs_assets ? "1" : "0");
@@ -460,6 +461,8 @@ DCPContent::identifier () const
                s += i->identifier () + " ";
        }
 
+       boost::mutex::scoped_lock lm(_mutex);
+
        s += string (_reference_video ? "1" : "0");
        for (auto text: _reference_text) {
                s += string(text ? "1" : "0");
@@ -717,13 +720,13 @@ DCPContent::can_reference_audio (shared_ptr<const Film> film, string& why_not) c
                return false;
        }
 
-        for (auto i: decoder->reels()) {
-                if (!i->main_sound()) {
-                       /// TRANSLATORS: this string will follow "Cannot reference this DCP: "
-                        why_not = _("it does not have sound in all its reels.");
-                        return false;
-                }
-        }
+       if (audio && audio->stream()) {
+               auto const channels = audio->stream()->channels();
+               if (channels != film->audio_channels()) {
+                       why_not = String::compose(_("it has a different number of audio channels than the project; set the project to have %1 channels."), channels);
+                       return false;
+               }
+       }
 
        /// TRANSLATORS: this string will follow "Cannot reference this DCP: "
        return can_reference(
@@ -745,6 +748,9 @@ DCPContent::can_reference_text (shared_ptr<const Film> film, TextType type, stri
        } catch (dcp::ReadError &) {
                /* We couldn't read the DCP, so it's probably missing */
                return false;
+       } catch (DCPError &) {
+               /* We couldn't read the DCP, so it's probably missing */
+               return false;
        } catch (dcp::KDMDecryptionError &) {
                /* We have an incorrect KDM */
                return false;
@@ -752,22 +758,13 @@ DCPContent::can_reference_text (shared_ptr<const Film> film, TextType type, stri
 
         for (auto i: decoder->reels()) {
                 if (type == TextType::OPEN_SUBTITLE) {
-                       if (!i->main_subtitle()) {
-                               /// TRANSLATORS: this string will follow "Cannot reference this DCP: "
-                               why_not = _("it does not have open subtitles in all its reels.");
-                               return false;
-                       } else if (i->main_subtitle()->entry_point().get_value_or(0) != 0) {
+                       if (i->main_subtitle() && i->main_subtitle()->entry_point().get_value_or(0) != 0) {
                                /// TRANSLATORS: this string will follow "Cannot reference this DCP: "
                                why_not = _("one of its subtitle reels has a non-zero entry point so it must be re-written.");
                                return false;
                        }
                 }
                if (type == TextType::CLOSED_CAPTION) {
-                       if (i->closed_captions().empty()) {
-                               /// TRANSLATORS: this string will follow "Cannot reference this DCP: "
-                               why_not = _("it does not have closed captions in all its reels.");
-                               return false;
-                       }
                        for (auto j: i->closed_captions()) {
                                if (j->entry_point().get_value_or(0) != 0) {
                                        /// TRANSLATORS: this string will follow "Cannot reference this DCP: "
@@ -787,8 +784,8 @@ DCPContent::can_reference_text (shared_ptr<const Film> film, TextType type, stri
        /// TRANSLATORS: this string will follow "Cannot reference this DCP: "
        return can_reference(
                film,
-               [](shared_ptr<const Content> c) {
-                       return !c->text.empty();
+               [type](shared_ptr<const Content> c) {
+                       return std::find_if(c->text.begin(), c->text.end(), [type](shared_ptr<const TextContent> t) { return t->type() == type; }) != c->text.end();
                },
                _("they overlap other text content; remove the other content."),
                why_not
@@ -803,6 +800,13 @@ DCPContent::take_settings_from (shared_ptr<const Content> c)
                return;
        }
 
+       if (this == dc.get()) {
+               return;
+       }
+
+       boost::mutex::scoped_lock lm(_mutex);
+       boost::mutex::scoped_lock lm2(dc->_mutex);
+
        _reference_video = dc->_reference_video;
        _reference_audio = dc->_reference_audio;
        _reference_text = dc->_reference_text;
@@ -834,7 +838,7 @@ DCPContent::kdm_timing_window_valid () const
 Resolution
 DCPContent::resolution () const
 {
-       if (video->size().width > 2048 || video->size().height > 1080) {
+       if (video->size() && (video->size()->width > 2048 || video->size()->height > 1080)) {
                return Resolution::FOUR_K;
        }
 
@@ -842,33 +846,6 @@ DCPContent::resolution () const
 }
 
 
-void
-add_fonts_from_examiner(shared_ptr<TextContent> text, vector<vector<shared_ptr<Font>>> const & all_fonts)
-{
-       int reel_number = 0;
-       for (auto reel_fonts: all_fonts) {
-               for (auto font: reel_fonts) {
-                       /* Each reel could have its own font with the same ID, so we disambiguate them here
-                        * by prepending the reel number.  We do the same disambiguation when emitting the
-                        * subtitles in the DCP decoder.
-                        */
-                       auto font_copy = make_shared<dcpomatic::Font>(*font);
-                       font_copy->set_id(id_for_font_in_reel(font->id(), reel_number));
-                       text->add_font(font_copy);
-               }
-               ++reel_number;
-       }
-
-}
-
-
-string
-id_for_font_in_reel(string id, int reel)
-{
-       return String::compose("%1_%2", reel, id);
-}
-
-
 void
 DCPContent::check_font_ids()
 {
@@ -877,7 +854,7 @@ DCPContent::check_font_ids()
        }
 
        DCPExaminer examiner(shared_from_this(), true);
-       add_fonts_from_examiner(text.front(), examiner.fonts());
+       examiner.add_fonts(text.front());
 }