Fix by-video-content reel split when there is content after the last video content.

diff --git a/ChangeLog b/ChangeLog
index 1a604dca16cd787c24a4cb6f30b99d5ff521bcef..9c36429f880576f3ef43bba0eaa3d7c570ecf2a8 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
 2015-10-19  Carl Hetherington  <cth@carlh.net>
 
+       * Fix by-video-content reel split when there is stuff
+       after the last piece of video content.
+
        * Add button to export leaf private key from the config.
 
 2015-10-17  Carl Hetherington  <cth@carlh.net>

diff --git a/src/lib/film.cc b/src/lib/film.cc
index ace180d1d24bab92eef6fc568e925bed98a2e668..47bcc35df5e5f4d462f1587dbd02b42132556dbf 100644 (file)
--- a/src/lib/film.cc
+++ b/src/lib/film.cc
@@ -1299,20 +1299,31 @@ Film::reels () const
                break;
        case REELTYPE_BY_VIDEO_CONTENT:
        {
-               optional<DCPTime> last;
+               optional<DCPTime> last_split;
+               shared_ptr<VideoContent> last_video;
+               ContentList cl = content ();
                BOOST_FOREACH (shared_ptr<Content> c, content ()) {
                        shared_ptr<VideoContent> v = dynamic_pointer_cast<VideoContent> (c);
                        if (v) {
                                BOOST_FOREACH (DCPTime t, v->reel_split_points()) {
-                                       if (last) {
-                                               p.push_back (DCPTimePeriod (last.get(), t));
+                                       if (last_split) {
+                                               p.push_back (DCPTimePeriod (last_split.get(), t));
                                        }
-                                       last = t;
+                                       last_split = t;
                                }
+                               last_video = v;
                        }
                }
-               if (last) {
-                       p.push_back (DCPTimePeriod (last.get(), len));
+
+               DCPTime video_end = last_video ? last_video->end() : DCPTime(0);
+               if (last_split) {
+                       /* Definitely go from the last split to the end of the video content */
+                       p.push_back (DCPTimePeriod (last_split.get(), video_end));
+               }
+
+               if (video_end < len) {
+                       /* And maybe go after that as well if there is any non-video hanging over the end */
+                       p.push_back (DCPTimePeriod (video_end, len));
                }
                break;
        }

diff --git a/src/lib/playlist.cc b/src/lib/playlist.cc
index 6e1d345721ea1e811df2d528044caf282a5f08a3..c9de9b88fdd02726ebd41aa0754b5f26501cb16e 100644 (file)
--- a/src/lib/playlist.cc
+++ b/src/lib/playlist.cc
@@ -351,7 +351,7 @@ ContentSorter::operator() (shared_ptr<Content> a, shared_ptr<Content> b)
        return a->position() < b->position();
 }
 
-/** @return content in an undefined order */
+/** @return content in ascending order of position */
 ContentList
 Playlist::content () const
 {

diff --git a/test/reels_test.cc b/test/reels_test.cc
index 85a9d07a9e236fc9bfe6e437656f10e6ee17f2df..3866c50bc5269f17dacf2609dc332a6d63b4a49a 100644 (file)
--- a/test/reels_test.cc
+++ b/test/reels_test.cc
@@ -23,8 +23,10 @@
 #include "lib/image_content.h"
 #include "lib/dcp_content_type.h"
 #include "lib/dcp_content.h"
+#include "lib/subrip_content.h"
 #include "test.h"
 #include <boost/test/unit_test.hpp>
+#include <boost/foreach.hpp>
 
 using std::list;
 using boost::shared_ptr;
@@ -111,8 +113,8 @@ BOOST_AUTO_TEST_CASE (reels_test2)
 
        check_dcp ("test/data/reels_test2", film->dir (film->dcp_name()));
 
-       shared_ptr<Film> film2 = new_test_film ("reels_test3");
-       film2->set_name ("reels_test3");
+       shared_ptr<Film> film2 = new_test_film ("reels_test2b");
+       film2->set_name ("reels_test2b");
        film2->set_container (Ratio::from_id ("185"));
        film2->set_dcp_content_type (DCPContentType::from_pretty_name ("Test"));
        film2->set_reel_type (REELTYPE_BY_VIDEO_CONTENT);
@@ -139,3 +141,39 @@ BOOST_AUTO_TEST_CASE (reels_test2)
        film2->make_dcp ();
        wait_for_jobs ();
 }
+
+/** Check that REELTYPE_BY_VIDEO_CONTENT adds an extra reel, if necessary, at the end
+ *  of all the video content to mop up anything afterward.
+ */
+BOOST_AUTO_TEST_CASE (reels_test3)
+{
+       shared_ptr<Film> film = new_test_film ("reels_test3");
+       film->set_name ("reels_test3");
+       film->set_container (Ratio::from_id ("185"));
+       film->set_dcp_content_type (DCPContentType::from_pretty_name ("Test"));
+       film->set_reel_type (REELTYPE_BY_VIDEO_CONTENT);
+
+       shared_ptr<Content> dcp (new DCPContent (film, "test/data/reels_test2"));
+       film->examine_and_add_content (dcp);
+       shared_ptr<Content> sub (new SubRipContent (film, "test/data/subrip.srt"));
+       film->examine_and_add_content (sub);
+       wait_for_jobs ();
+
+       std::cout << dcp->position() << " " << dcp->full_length() << "\n";
+       std::cout << sub->position() << " " << sub->full_length() << "\n";
+
+       list<DCPTimePeriod> reels = film->reels();
+       BOOST_REQUIRE_EQUAL (reels.size(), 4);
+       list<DCPTimePeriod>::const_iterator i = reels.begin ();
+       BOOST_CHECK_EQUAL (i->from, DCPTime (0));
+       BOOST_CHECK_EQUAL (i->to, DCPTime (96000));
+       ++i;
+       BOOST_CHECK_EQUAL (i->from, DCPTime (96000));
+       BOOST_CHECK_EQUAL (i->to, DCPTime (96000 * 2));
+       ++i;
+       BOOST_CHECK_EQUAL (i->from, DCPTime (96000 * 2));
+       BOOST_CHECK_EQUAL (i->to, DCPTime (96000 * 3));
+       ++i;
+       BOOST_CHECK_EQUAL (i->from, DCPTime (96000 * 3));
+       BOOST_CHECK_EQUAL (i->to, sub->full_length());
+}