X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Flib%2Fimage_content.cc;h=bb2de93073e42cc8b88a040a34c9163bd3e7de74;hb=e46302d86c7295bd95cd7cdfa331c8186fe793cb;hp=20f32c05dc0310c6d14b2a5d30758711c8218e1d;hpb=dd9be86db6cde0afa5da0d1d1ac43b42e05dca26;p=dcpomatic.git

diff --git a/src/lib/image_content.cc b/src/lib/image_content.cc
index 20f32c05d..bb2de9307 100644
--- a/src/lib/image_content.cc
+++ b/src/lib/image_content.cc
@@ -18,32 +18,36 @@
 
 */
 
-#include "image_content.h"
-#include "video_content.h"
-#include "image_examiner.h"
+
 #include "compose.hpp"
+#include "exceptions.h"
 #include "film.h"
-#include "job.h"
 #include "frame_rate_change.h"
-#include "exceptions.h"
+#include "image_content.h"
+#include "image_examiner.h"
 #include "image_filename_sorter.h"
+#include "job.h"
+#include "util.h"
+#include "video_content.h"
 #include <libcxml/cxml.h>
 #include <libxml++/libxml++.h>
-#include <boost/foreach.hpp>
 #include <iostream>
 
 #include "i18n.h"
 
-using std::string;
+
 using std::cout;
 using std::list;
-using std::vector;
+using std::make_shared;
 using std::shared_ptr;
+using std::string;
+using std::vector;
 using namespace dcpomatic;
 
+
 ImageContent::ImageContent (boost::filesystem::path p)
 {
-	video.reset (new VideoContent (this));
+	video = make_shared<VideoContent>(this);
 
 	if (boost::filesystem::is_regular_file (p) && valid_image_file (p)) {
 		add_path (p);
@@ -58,9 +62,10 @@ ImageContent::ImageContent (boost::filesystem::path p)
 ImageContent::ImageContent (cxml::ConstNodePtr node, int version)
 	: Content (node)
 {
-	video = VideoContent::from_xml (this, node, version);
+	video = VideoContent::from_xml (this, node, version, VideoRange::FULL);
 }
 
+
 string
 ImageContent::summary () const
 {
@@ -75,6 +80,7 @@ ImageContent::summary () const
 	return s;
 }
 
+
 string
 ImageContent::technical_summary () const
 {
@@ -90,6 +96,7 @@ ImageContent::technical_summary () const
 	return s;
 }
 
+
 void
 ImageContent::as_xml (xmlpp::Node* node, bool with_paths) const
 {
@@ -101,6 +108,7 @@ ImageContent::as_xml (xmlpp::Node* node, bool with_paths) const
 	}
 }
 
+
 void
 ImageContent::examine (shared_ptr<const Film> film, shared_ptr<Job> job)
 {
@@ -108,9 +116,9 @@ ImageContent::examine (shared_ptr<const Film> film, shared_ptr<Job> job)
 		job->sub (_("Scanning image files"));
 		vector<boost::filesystem::path> paths;
 		int n = 0;
-		for (boost::filesystem::directory_iterator i(*_path_to_scan); i != boost::filesystem::directory_iterator(); ++i) {
-			if (boost::filesystem::is_regular_file (i->path()) && valid_image_file (i->path())) {
-				paths.push_back (i->path());
+		for (auto i: boost::filesystem::directory_iterator(*_path_to_scan)) {
+			if (boost::filesystem::is_regular_file(i.path()) && valid_image_file (i.path())) {
+				paths.push_back (i.path());
 			}
 			++n;
 			if ((n % 1000) == 0) {
@@ -128,11 +136,12 @@ ImageContent::examine (shared_ptr<const Film> film, shared_ptr<Job> job)
 
 	Content::examine (film, job);
 
-	shared_ptr<ImageExaminer> examiner (new ImageExaminer (film, shared_from_this(), job));
-	video->take_from_examiner (examiner);
+	auto examiner = make_shared<ImageExaminer>(film, shared_from_this(), job);
+	video->take_from_examiner(film, examiner);
 	set_default_colour_conversion ();
 }
 
+
 DCPTime
 ImageContent::full_length (shared_ptr<const Film> film) const
 {
@@ -140,12 +149,14 @@ ImageContent::full_length (shared_ptr<const Film> film) const
 	return DCPTime::from_frames (llrint(video->length_after_3d_combine() * frc.factor()), film->video_frame_rate());
 }
 
+
 DCPTime
 ImageContent::approximate_length () const
 {
 	return DCPTime::from_frames (video->length_after_3d_combine(), 24);
 }
 
+
 string
 ImageContent::identifier () const
 {
@@ -154,16 +165,18 @@ ImageContent::identifier () const
 	return buffer;
 }
 
+
 bool
 ImageContent::still () const
 {
 	return number_of_paths() == 1;
 }
 
+
 void
 ImageContent::set_default_colour_conversion ()
 {
-	BOOST_FOREACH (boost::filesystem::path i, paths()) {
+	for (auto i: paths()) {
 		if (valid_j2k_file (i)) {
 			/* We default to no colour conversion if we have JPEG2000 files */
 			video->unset_colour_conversion ();
@@ -182,6 +195,7 @@ ImageContent::set_default_colour_conversion ()
 	}
 }
 
+
 void
 ImageContent::add_properties (shared_ptr<const Film> film, list<UserProperty>& p) const
 {