X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Flib%2Fcontent.cc;h=9b16eff7fa548d19ff82a4ea529138e375dc12f2;hb=b733ff51f0212d02dbf33ccb62e67f07941f5ace;hp=21b04c6493aea2f3d2ee0410a788b39a8dc85d04;hpb=fe56f4e0ac26fbe1bea8d7b07f61dcc26cc26984;p=dcpomatic.git

diff --git a/src/lib/content.cc b/src/lib/content.cc
index 21b04c649..9b16eff7f 100644
--- a/src/lib/content.cc
+++ b/src/lib/content.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2013-2016 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2013-2018 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -27,7 +27,7 @@
 #include "content_factory.h"
 #include "video_content.h"
 #include "audio_content.h"
-#include "subtitle_content.h"
+#include "text_content.h"
 #include "exceptions.h"
 #include "film.h"
 #include "job.h"
@@ -48,6 +48,7 @@ using std::vector;
 using std::max;
 using std::pair;
 using boost::shared_ptr;
+using boost::optional;
 using dcp::raw_convert;
 using dcp::locale_convert;
 
@@ -190,6 +191,16 @@ Content::signal_changed (int p)
 void
 Content::set_position (DCPTime p)
 {
+	/* video and audio content can modify its position */
+
+	if (video) {
+		video->modify_position (p);
+	}
+
+	if (audio) {
+		audio->modify_position (p);
+	}
+
 	{
 		boost::mutex::scoped_lock lm (_mutex);
 		if (p == _position) {
@@ -205,6 +216,16 @@ Content::set_position (DCPTime p)
 void
 Content::set_trim_start (ContentTime t)
 {
+	/* video and audio content can modify its start trim */
+
+	if (video) {
+		video->modify_trim_start (t);
+	}
+
+	if (audio) {
+		audio->modify_trim_start (t);
+	}
+
 	{
 		boost::mutex::scoped_lock lm (_mutex);
 		_trim_start = t;
@@ -333,8 +354,10 @@ list<DCPTime>
 Content::reel_split_points () const
 {
 	list<DCPTime> t;
-	/* XXX: this is questionable; perhaps the position itself should be forced to be on a frame boundary */
-	t.push_back (position().ceil (film()->video_frame_rate()));
+	/* This is only called for video content and such content has its position forced
+	   to start on a frame boundary.
+	*/
+	t.push_back (position());
 	return t;
 }
 
@@ -347,6 +370,23 @@ Content::set_video_frame_rate (double r)
 	}
 
 	signal_changed (ContentProperty::VIDEO_FRAME_RATE);
+
+	/* Make sure things are still on frame boundaries */
+	if (video) {
+		set_position (position());
+		set_trim_start (trim_start());
+	}
+}
+
+void
+Content::unset_video_frame_rate ()
+{
+	{
+		boost::mutex::scoped_lock lm (_mutex);
+		_video_frame_rate = optional<double>();
+	}
+
+	signal_changed (ContentProperty::VIDEO_FRAME_RATE);
 }
 
 double
@@ -396,16 +436,44 @@ Content::add_properties (list<UserProperty>& p) const
 	}
 }
 
+/** Take settings from the given content if it is of the correct type */
 void
-Content::use_template (shared_ptr<const Content> c)
+Content::take_settings_from (shared_ptr<const Content> c)
 {
 	if (video && c->video) {
-		video->use_template (c->video);
+		video->take_settings_from (c->video);
 	}
 	if (audio && c->audio) {
-		audio->use_template (c->audio);
+		audio->take_settings_from (c->audio);
+	}
+
+	list<shared_ptr<TextContent> >::iterator i = text.begin ();
+	list<shared_ptr<TextContent> >::const_iterator j = c->text.begin ();
+	while (i != text.end() && j != c->text.end()) {
+		(*i)->take_settings_from (*j);
+		++i;
+		++j;
+ 	}
+}
+
+shared_ptr<TextContent>
+Content::only_text () const
+{
+	DCPOMATIC_ASSERT (text.size() < 2);
+	if (text.empty ()) {
+		return shared_ptr<TextContent> ();
 	}
-	if (subtitle && c->subtitle) {
-		subtitle->use_template (c->subtitle);
+	return text.front ();
+}
+
+shared_ptr<TextContent>
+Content::text_of_original_type (TextType type) const
+{
+	BOOST_FOREACH (shared_ptr<TextContent> i, text) {
+		if (i->original_type() == type) {
+			return i;
+		}
 	}
+
+	return shared_ptr<TextContent> ();
 }