Trim referred-to DCPs correctly using CPL entry point / duration.

diff --git a/ChangeLog b/ChangeLog
index 4f22c362e27363d31d9e5ff5528d279d34e63675..b080ee1d49c76e26ea235d26d8e1d6733e5306af 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2016-09-21  Carl Hetherington  <cth@carlh.net>
+
+       * Fix trimming of OV DCPs that are referred to (#956).
+
 2016-09-20  Carl Hetherington  <cth@carlh.net>
 
        * Give better errors on SubRip parse failures.

diff --git a/cscript b/cscript
index 57d46737da54a9b874cb5c63e63b18b11125fcee..ef67e544ffad96120bd645f2632263a3c79c634a 100644 (file)
--- a/cscript
+++ b/cscript
@@ -233,7 +233,7 @@ def dependencies(target):
         ffmpeg_options = {}
 
     return (('ffmpeg-cdist', '1d4a1a4', ffmpeg_options),
-            ('libdcp', 'b105b7d'),
+            ('libdcp', 'd6fda12'),
             ('libsub', '3db5052'))
 
 def configure_options(target):

diff --git a/src/lib/player.cc b/src/lib/player.cc
index 1506436e1ebcf75d6b1d72b83dc077b91c582cad..c17cdfdc0166d04fd376024e40a7494aa85215e0 100644 (file)
--- a/src/lib/player.cc
+++ b/src/lib/player.cc
@@ -739,8 +739,17 @@ Player::get_reel_assets ()
 
                int64_t offset = 0;
                BOOST_FOREACH (shared_ptr<dcp::Reel> k, decoder->reels()) {
+
+                       DCPOMATIC_ASSERT (j->video_frame_rate ());
+                       double const cfr = j->video_frame_rate().get();
+                       Frame const trim_start = j->trim_start().frames_round (cfr);
+                       Frame const trim_end = j->trim_end().frames_round (cfr);
+
                        DCPTime const from = i->position() + DCPTime::from_frames (offset, _film->video_frame_rate());
                        if (j->reference_video ()) {
+                               DCPOMATIC_ASSERT (k->main_picture ());
+                               k->main_picture()->set_entry_point (trim_start);
+                               k->main_picture()->set_duration (k->main_picture()->intrinsic_duration() - trim_start - trim_end);
                                a.push_back (
                                        ReferencedReelAsset (
                                                k->main_picture (),
@@ -750,6 +759,9 @@ Player::get_reel_assets ()
                        }
 
                        if (j->reference_audio ()) {
+                               DCPOMATIC_ASSERT (k->main_sound ());
+                               k->main_sound()->set_entry_point (trim_start);
+                               k->main_sound()->set_duration (k->main_sound()->intrinsic_duration() - trim_start - trim_end);
                                a.push_back (
                                        ReferencedReelAsset (
                                                k->main_sound (),
@@ -760,6 +772,8 @@ Player::get_reel_assets ()
 
                        if (j->reference_subtitle ()) {
                                DCPOMATIC_ASSERT (k->main_subtitle ());
+                               k->main_subtitle()->set_entry_point (trim_start);
+                               k->main_subtitle()->set_duration (k->main_subtitle()->intrinsic_duration() - trim_start - trim_end);
                                a.push_back (
                                        ReferencedReelAsset (
                                                k->main_subtitle (),

diff --git a/test/vf_test.cc b/test/vf_test.cc
index 4bfb2ad6e5a7633ecca2616433fc00d51bd4cff2..ab39a651043634623bc884ff4f77fb5dcaa3e071 100644 (file)
--- a/test/vf_test.cc
+++ b/test/vf_test.cc
@@ -131,3 +131,40 @@ BOOST_AUTO_TEST_CASE (vf_test2)
        BOOST_CHECK_EQUAL (vf_c.cpls().front()->reels().front()->main_sound()->id(), sound_id);
        BOOST_REQUIRE (vf_c.cpls().front()->reels().front()->main_subtitle());
 }
+
+/** Test creation of a VF using a trimmed OV; the output should have entry point /
+ *  duration altered to effect the trimming.
+ */
+BOOST_AUTO_TEST_CASE (vf_test3)
+{
+       /* Make the OV */
+       shared_ptr<Film> ov = new_test_film ("vf_test3_ov");
+       ov->set_dcp_content_type (DCPContentType::from_isdcf_name ("TST"));
+       ov->set_name ("vf_test3_ov");
+       shared_ptr<Content> video = content_factory (ov, "test/data/flat_red.png");
+       ov->examine_and_add_content (video);
+       wait_for_jobs ();
+       video->video->set_length (24 * 5);
+       shared_ptr<Content> audio = content_factory (ov, "test/data/white.wav");
+       ov->examine_and_add_content (audio);
+       wait_for_jobs ();
+       ov->make_dcp ();
+       wait_for_jobs ();
+
+       /* Make the VF */
+       shared_ptr<Film> vf = new_test_film ("vf_test3_vf");
+       vf->set_name ("vf_test3_vf");
+       vf->set_dcp_content_type (DCPContentType::from_isdcf_name ("TST"));
+       vf->set_reel_type (REELTYPE_BY_VIDEO_CONTENT);
+       shared_ptr<DCPContent> dcp = dynamic_pointer_cast<DCPContent> (content_factory (vf, ov->dir (ov->dcp_name ())));
+       BOOST_REQUIRE (dcp);
+       dcp->set_trim_start (ContentTime::from_seconds (1));
+       dcp->set_trim_end (ContentTime::from_seconds (1));
+       vf->examine_and_add_content (dcp);
+       wait_for_jobs ();
+       dcp->set_reference_video (true);
+       dcp->set_reference_audio (true);
+       vf->make_dcp ();
+       wait_for_jobs ();
+       vf->write_metadata ();
+}