From ca56871d4860b8ead384c410fe374c2fa993f88f Mon Sep 17 00:00:00 2001
From: Carl Hetherington <cth@carlh.net>
Date: Mon, 4 May 2020 23:02:45 +0200
Subject: Add KDMWithMetadataPtr typedef

---
 src/lib/cinema_kdms.cc       | 14 +++++++-------
 src/lib/cinema_kdms.h        |  4 ++--
 src/lib/kdm_with_metadata.cc |  8 ++++----
 src/lib/kdm_with_metadata.h  | 17 +++++++++++------
 4 files changed, 24 insertions(+), 19 deletions(-)

(limited to 'src/lib')

diff --git a/src/lib/cinema_kdms.cc b/src/lib/cinema_kdms.cc
index 99d151152..61234ff3f 100644
--- a/src/lib/cinema_kdms.cc
+++ b/src/lib/cinema_kdms.cc
@@ -47,7 +47,7 @@ CinemaKDMs::make_zip_file (boost::filesystem::path zip_file, dcp::NameFormat nam
 
 	name_values['c'] = cinema->name;
 
-	BOOST_FOREACH (shared_ptr<KDMWithMetadata> i, screen_kdms) {
+	BOOST_FOREACH (KDMWithMetadataPtr i, screen_kdms) {
 		name_values['s'] = i->screen->name;
 		name_values['i'] = i->kdm_id ();
 		string const name = careful_string_filter(name_format.get(name_values, ".xml"));
@@ -61,7 +61,7 @@ CinemaKDMs::make_zip_file (boost::filesystem::path zip_file, dcp::NameFormat nam
  *  CinemaKDM contains the KDMs for its cinema.
  */
 list<CinemaKDMs>
-CinemaKDMs::collect (list<shared_ptr<KDMWithMetadata> > screen_kdms)
+CinemaKDMs::collect (list<KDMWithMetadataPtr> screen_kdms)
 {
 	list<CinemaKDMs> cinema_kdms;
 
@@ -71,17 +71,17 @@ CinemaKDMs::collect (list<shared_ptr<KDMWithMetadata> > screen_kdms)
 
 		CinemaKDMs ck;
 
-		list<shared_ptr<KDMWithMetadata> >::iterator i = screen_kdms.begin ();
+		list<KDMWithMetadataPtr>::iterator i = screen_kdms.begin ();
 		ck.cinema = (*i)->screen->cinema;
 		ck.screen_kdms.push_back (*i);
-		list<shared_ptr<KDMWithMetadata> >::iterator j = i;
+		list<KDMWithMetadataPtr>::iterator j = i;
 		++i;
 		screen_kdms.remove (*j);
 
 		while (i != screen_kdms.end ()) {
 			if ((*i)->screen->cinema == ck.cinema) {
 				ck.screen_kdms.push_back (*i);
-				list<shared_ptr<KDMWithMetadata> >::iterator j = i;
+				list<KDMWithMetadataPtr>::iterator j = i;
 				++i;
 				screen_kdms.remove (*j);
 			} else {
@@ -117,7 +117,7 @@ CinemaKDMs::write_directories (
 		path /= container_name_format.get(name_values, "");
 		if (!boost::filesystem::exists (path) || confirm_overwrite (path)) {
 			boost::filesystem::create_directories (path);
-			KDMWithMetadata::write_files (i.screen_kdms, path, filename_format, name_values, confirm_overwrite);
+			write_files (i.screen_kdms, path, filename_format, name_values, confirm_overwrite);
 		}
 		written += i.screen_kdms.size();
 	}
@@ -209,7 +209,7 @@ CinemaKDMs::email (
 		boost::algorithm::replace_all (body, "$CINEMA_NAME", i.cinema->name);
 
 		string screens;
-		BOOST_FOREACH (shared_ptr<KDMWithMetadata> j, i.screen_kdms) {
+		BOOST_FOREACH (KDMWithMetadataPtr j, i.screen_kdms) {
 			screens += j->screen->name + ", ";
 		}
 		boost::algorithm::replace_all (body, "$SCREENS", screens.substr (0, screens.length() - 2));
diff --git a/src/lib/cinema_kdms.h b/src/lib/cinema_kdms.h
index 0626ded5d..918670397 100644
--- a/src/lib/cinema_kdms.h
+++ b/src/lib/cinema_kdms.h
@@ -29,7 +29,7 @@ class CinemaKDMs
 public:
 	void make_zip_file (boost::filesystem::path zip_file, dcp::NameFormat name_format, dcp::NameFormat::Map name_values) const;
 
-	static std::list<CinemaKDMs> collect (std::list<boost::shared_ptr<KDMWithMetadata> > kdms);
+	static std::list<CinemaKDMs> collect (std::list<KDMWithMetadataPtr > kdms);
 
 	static int write_directories (
 		std::list<CinemaKDMs> cinema_kdms,
@@ -58,5 +58,5 @@ public:
 		);
 
 	boost::shared_ptr<Cinema> cinema;
-	std::list<boost::shared_ptr<KDMWithMetadata> > screen_kdms;
+	std::list<KDMWithMetadataPtr > screen_kdms;
 };
diff --git a/src/lib/kdm_with_metadata.cc b/src/lib/kdm_with_metadata.cc
index e6b283415..674554f7e 100644
--- a/src/lib/kdm_with_metadata.cc
+++ b/src/lib/kdm_with_metadata.cc
@@ -30,8 +30,8 @@ using std::list;
 using boost::shared_ptr;
 
 int
-KDMWithMetadata::write_files (
-	list<shared_ptr<KDMWithMetadata> > screen_kdms,
+write_files (
+	list<KDMWithMetadataPtr> screen_kdms,
 	boost::filesystem::path directory,
 	dcp::NameFormat name_format,
 	dcp::NameFormat::Map name_values,
@@ -42,7 +42,7 @@ KDMWithMetadata::write_files (
 
 	if (directory == "-") {
 		/* Write KDMs to the stdout */
-		BOOST_FOREACH (shared_ptr<KDMWithMetadata> i, screen_kdms) {
+		BOOST_FOREACH (KDMWithMetadataPtr i, screen_kdms) {
 			cout << i->kdm_as_xml ();
 			++written;
 		}
@@ -55,7 +55,7 @@ KDMWithMetadata::write_files (
 	}
 
 	/* Write KDMs to the specified directory */
-	BOOST_FOREACH (shared_ptr<KDMWithMetadata> i, screen_kdms) {
+	BOOST_FOREACH (KDMWithMetadataPtr i, screen_kdms) {
 		name_values['c'] = i->screen->cinema ? i->screen->cinema->name : "";
 		name_values['s'] = i->screen->name;
 		name_values['i'] = i->kdm_id ();
diff --git a/src/lib/kdm_with_metadata.h b/src/lib/kdm_with_metadata.h
index 8d6d61daa..19af0356f 100644
--- a/src/lib/kdm_with_metadata.h
+++ b/src/lib/kdm_with_metadata.h
@@ -46,15 +46,20 @@ public:
 	virtual void kdm_as_xml (boost::filesystem::path out) const = 0;
 	virtual std::string kdm_id () const = 0;
 
-	static int write_files (
-		std::list<boost::shared_ptr<KDMWithMetadata> > screen_kdms, boost::filesystem::path directory,
-		dcp::NameFormat name_format, dcp::NameFormat::Map name_values,
-		boost::function<bool (boost::filesystem::path)> confirm_overwrite
-		);
-
 	boost::shared_ptr<dcpomatic::Screen> screen;
 };
 
+
+typedef boost::shared_ptr<KDMWithMetadata> KDMWithMetadataPtr;
+
+
+int write_files (
+	std::list<KDMWithMetadataPtr> screen_kdms, boost::filesystem::path directory,
+	dcp::NameFormat name_format, dcp::NameFormat::Map name_values,
+	boost::function<bool (boost::filesystem::path)> confirm_overwrite
+	);
+
+
 class DCPKDMWithMetadata : public KDMWithMetadata
 {
 public:
-- 
cgit v1.2.3