Fix incorrect reels when the first content is not at time 0.

With REELTYPE_BY_VIDEO_CONTENT the first reel would not start at 0.

diff --git a/src/lib/film.cc b/src/lib/film.cc
index f0dc683ed2587819951ce9c2a2e164198ed70752..d19dcc73c3d5b8377e2258eeffeb93c361cf7918 100644 (file)
--- a/src/lib/film.cc
+++ b/src/lib/film.cc
@@ -1606,24 +1606,24 @@ Film::reels () const
                break;
        case REELTYPE_BY_VIDEO_CONTENT:
        {
                break;
        case REELTYPE_BY_VIDEO_CONTENT:
        {

-               optional<DCPTime> last_split;
+               DCPTime last_split;

                shared_ptr<Content> last_video;
                BOOST_FOREACH (shared_ptr<Content> c, content ()) {
                        if (c->video) {
                                BOOST_FOREACH (DCPTime t, c->reel_split_points(shared_from_this())) {
                shared_ptr<Content> last_video;
                BOOST_FOREACH (shared_ptr<Content> c, content ()) {
                        if (c->video) {
                                BOOST_FOREACH (DCPTime t, c->reel_split_points(shared_from_this())) {

-                                       if (last_split) {
-                                               p.push_back (DCPTimePeriod (last_split.get(), t));
+                                       if (last_split != t) {
+                                               p.push_back (DCPTimePeriod(last_split, t));
+                                               last_split = t;

                                        }
                                        }

-                                       last_split = t;

                                }
                                last_video = c;
                        }
                }
 
                DCPTime video_end = last_video ? last_video->end(shared_from_this()) : DCPTime(0);
                                }
                                last_video = c;
                        }
                }
 
                DCPTime video_end = last_video ? last_video->end(shared_from_this()) : DCPTime(0);

-               if (last_split) {
+               if (last_split != video_end) {

                        /* Definitely go from the last split to the end of the video content */
                        /* Definitely go from the last split to the end of the video content */

-                       p.push_back (DCPTimePeriod (last_split.get(), video_end));
+                       p.push_back (DCPTimePeriod(last_split, video_end));

                }
 
                if (video_end < len) {
                }
 
                if (video_end < len) {

diff --git a/test/reels_test.cc b/test/reels_test.cc
index 620c934178723f21d421e458630686e23e4a9708..55561acca62f1d43eaa2850b1bad7a149126f5fb 100644 (file)
--- a/test/reels_test.cc
+++ b/test/reels_test.cc
@@ -1,5 +1,5 @@
 /*
 /*

-    Copyright (C) 2015-2016 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2015-2019 Carl Hetherington <cth@carlh.net>

 
     This file is part of DCP-o-matic.
 
 
     This file is part of DCP-o-matic.
 


@@ -35,6 +35,7 @@
 #include "test.h"
 #include <boost/test/unit_test.hpp>
 #include <boost/foreach.hpp>
 #include "test.h"
 #include <boost/test/unit_test.hpp>
 #include <boost/foreach.hpp>

+#include <iostream>

 
 using std::list;
 using std::cout;
 
 using std::list;
 using std::cout;


@@ -413,3 +414,30 @@ BOOST_AUTO_TEST_CASE (reels_test10)
        BOOST_REQUIRE (!wait_for_jobs());
        vf->write_metadata ();
 }
        BOOST_REQUIRE (!wait_for_jobs());
        vf->write_metadata ();
 }

+
+/** Another reels error; REELTYPE_BY_VIDEO_CONTENT when the first content is not
+ *  at time 0.
+ */
+BOOST_AUTO_TEST_CASE (reels_test11)
+{
+       shared_ptr<Film> film = new_test_film2 ("reels_test11");
+       film->set_video_frame_rate (24);
+       shared_ptr<FFmpegContent> A(new FFmpegContent("test/data/flat_red.png"));
+       film->examine_and_add_content (A);
+       BOOST_REQUIRE (!wait_for_jobs());
+       A->video->set_length (240);
+       A->set_video_frame_rate (24);
+       A->set_position (film, DCPTime::from_seconds(1));
+       film->set_reel_type (REELTYPE_BY_VIDEO_CONTENT);
+       film->make_dcp ();
+       BOOST_REQUIRE (!wait_for_jobs());
+       BOOST_CHECK_EQUAL (A->position().get(), DCPTime::from_seconds(1).get());
+       BOOST_CHECK_EQUAL (A->end(film).get(), DCPTime::from_seconds(1 + 10).get());
+
+       list<DCPTimePeriod> r = film->reels ();
+       BOOST_CHECK_EQUAL (r.size(), 2);
+       BOOST_CHECK_EQUAL (r.front().from.get(), 0);
+       BOOST_CHECK_EQUAL (r.front().to.get(), DCPTime::from_seconds(1).get());
+       BOOST_CHECK_EQUAL (r.back().from.get(), DCPTime::from_seconds(1).get());
+       BOOST_CHECK_EQUAL (r.back().to.get(), DCPTime::from_seconds(1 + 10).get());
+}