X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Flib%2Fffmpeg_content.cc;h=20fe7776bd9aafb80ca12c0fbd035875c0551aaf;hb=3144a348751dc0c91ba1d1c7fe6c90127de9ef21;hp=6681a4f0a897aa6c6c9de3c192c1cad434d69a8c;hpb=da38c8a0156808595be0aae1b1490069a663fb78;p=dcpomatic.git

diff --git a/src/lib/ffmpeg_content.cc b/src/lib/ffmpeg_content.cc
index 6681a4f0a..20fe7776b 100644
--- a/src/lib/ffmpeg_content.cc
+++ b/src/lib/ffmpeg_content.cc
@@ -115,9 +115,8 @@ FFmpegContent::FFmpegContent (cxml::ConstNodePtr node, int version, list<string>
 	}
 
 	for (auto i: node->node_children("Filter")) {
-		Filter const * f = Filter::from_id(i->content());
-		if (f) {
-			_filters.push_back (f);
+		if (auto filter = Filter::from_id(i->content())) {
+			_filters.push_back(*filter);
 		} else {
 			notes.push_back (String::compose (_("DCP-o-matic no longer supports the `%1' filter, so it has been turned off."), i->content()));
 		}
@@ -232,7 +231,7 @@ FFmpegContent::as_xml (xmlpp::Node* node, bool with_paths) const
 	}
 
 	for (auto i: _filters) {
-		node->add_child("Filter")->add_child_text(i->id());
+		node->add_child("Filter")->add_child_text(i.id());
 	}
 
 	if (_first_video) {
@@ -292,14 +291,17 @@ FFmpegContent::examine (shared_ptr<const Film> film, shared_ptr<Job> job)
 			if (examiner->rotation()) {
 				auto rot = *examiner->rotation ();
 				if (fabs (rot - 180) < 1.0) {
-					_filters.push_back (Filter::from_id ("vflip"));
-					_filters.push_back (Filter::from_id ("hflip"));
+					_filters.push_back(*Filter::from_id("vflip"));
+					_filters.push_back(*Filter::from_id("hflip"));
 				} else if (fabs (rot - 90) < 1.0) {
-					_filters.push_back (Filter::from_id ("90clock"));
+					_filters.push_back(*Filter::from_id("90clock"));
 				} else if (fabs (rot - 270) < 1.0) {
-					_filters.push_back (Filter::from_id ("90anticlock"));
+					_filters.push_back(*Filter::from_id("90anticlock"));
 				}
 			}
+			if (examiner->has_alpha()) {
+				_filters.push_back(*Filter::from_id("premultiply"));
+			}
 		}
 
 		if (!examiner->audio_streams().empty()) {
@@ -455,7 +457,7 @@ FFmpegContent::approximate_length () const
 
 
 void
-FFmpegContent::set_filters (vector<Filter const *> const & filters)
+FFmpegContent::set_filters(vector<Filter> const& filters)
 {
 	ContentChangeSignaller cc (this, FFmpegContentProperty::FILTERS);
 
@@ -486,7 +488,7 @@ FFmpegContent::identifier () const
 	}
 
 	for (auto i: _filters) {
-		s += "_" + i->id();
+		s += "_" + i.id();
 	}
 
 	return s;