From 47c34c6c7c82396b62e101283cb25b8726dbaefd Mon Sep 17 00:00:00 2001
From: Carl Hetherington <cth@carlh.net>
Date: Tue, 27 Mar 2018 23:28:27 +0100
Subject: why_not in can_reference can just be a string rather than a list of
 strings.

---
 src/lib/dcp_content.cc | 26 +++++++++++++-------------
 src/lib/dcp_content.h  | 10 +++++-----
 2 files changed, 18 insertions(+), 18 deletions(-)

(limited to 'src/lib')

diff --git a/src/lib/dcp_content.cc b/src/lib/dcp_content.cc
index ad489917d..c4f73e2b2 100644
--- a/src/lib/dcp_content.cc
+++ b/src/lib/dcp_content.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2014-2016 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2014-2018 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -429,22 +429,22 @@ DCPContent::reel_split_points () const
 }
 
 bool
-DCPContent::can_reference (function<shared_ptr<ContentPart> (shared_ptr<const Content>)> part, string overlapping, list<string>& why_not) const
+DCPContent::can_reference (function<shared_ptr<ContentPart> (shared_ptr<const Content>)> part, string overlapping, string& why_not) const
 {
 	/* We must be using the same standard as the film */
 	if (_standard) {
 		if (_standard.get() == dcp::INTEROP && !film()->interop()) {
-			why_not.push_back (_("The film is set to SMPTE and this DCP is Interop."));
+			why_not = _("The film is set to SMPTE and this DCP is Interop.");
 			return false;
 		} else if (_standard.get() == dcp::SMPTE && film()->interop()) {
-			why_not.push_back (_("The film is set to Interop and this DCP is SMPTE."));
+			why_not = _("The film is set to Interop and this DCP is SMPTE.");
 			return false;
 		}
 	}
 
 	/* And the same frame rate */
 	if (!video_frame_rate() || (lrint(video_frame_rate().get()) != film()->video_frame_rate())) {
-		why_not.push_back (_("The film has a different frame rate to this DCP."));
+		why_not = _("The film has a different frame rate to this DCP.");
 		return false;
 	}
 
@@ -466,14 +466,14 @@ DCPContent::can_reference (function<shared_ptr<ContentPart> (shared_ptr<const Co
 	*/
 	BOOST_FOREACH (DCPTimePeriod i, reel_list) {
 		if (find (fr.begin(), fr.end(), i) == fr.end ()) {
-			why_not.push_back (_("The reel lengths in the film differ from those in the DCP; set the reel mode to 'split by video content'."));
+			why_not = _("The reel lengths in the film differ from those in the DCP; set the reel mode to 'split by video content'.");
 			return false;
 		}
 	}
 
 	ContentList a = overlaps (film()->content(), part, position(), end());
 	if (a.size() != 1 || a.front().get() != this) {
-		why_not.push_back (overlapping);
+		why_not = overlapping;
 		return false;
 	}
 
@@ -481,10 +481,10 @@ DCPContent::can_reference (function<shared_ptr<ContentPart> (shared_ptr<const Co
 }
 
 bool
-DCPContent::can_reference_video (list<string>& why_not) const
+DCPContent::can_reference_video (string& why_not) const
 {
 	if (film()->frame_size() != video->size()) {
-		why_not.push_back (_("The video frame size in the film differs from that in the DCP."));
+		why_not = _("The video frame size in the film differs from that in the DCP.");
 		return false;
 	}
 
@@ -492,7 +492,7 @@ DCPContent::can_reference_video (list<string>& why_not) const
 }
 
 bool
-DCPContent::can_reference_audio (list<string>& why_not) const
+DCPContent::can_reference_audio (string& why_not) const
 {
 	shared_ptr<DCPDecoder> decoder;
 	try {
@@ -507,7 +507,7 @@ DCPContent::can_reference_audio (list<string>& why_not) const
 
         BOOST_FOREACH (shared_ptr<dcp::Reel> i, decoder->reels()) {
                 if (!i->main_sound()) {
-                        why_not.push_back (_("The DCP does not have sound in all reels."));
+                        why_not = _("The DCP does not have sound in all reels.");
                         return false;
                 }
         }
@@ -516,7 +516,7 @@ DCPContent::can_reference_audio (list<string>& why_not) const
 }
 
 bool
-DCPContent::can_reference_subtitle (list<string>& why_not) const
+DCPContent::can_reference_subtitle (string& why_not) const
 {
 	shared_ptr<DCPDecoder> decoder;
 	try {
@@ -531,7 +531,7 @@ DCPContent::can_reference_subtitle (list<string>& why_not) const
 
         BOOST_FOREACH (shared_ptr<dcp::Reel> i, decoder->reels()) {
                 if (!i->main_subtitle()) {
-                        why_not.push_back (_("The DCP does not have subtitles in all reels."));
+                        why_not = _("The DCP does not have subtitles in all reels.");
                         return false;
                 }
         }
diff --git a/src/lib/dcp_content.h b/src/lib/dcp_content.h
index 892b6aa73..371ec7fff 100644
--- a/src/lib/dcp_content.h
+++ b/src/lib/dcp_content.h
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2014-2016 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2014-2018 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -97,7 +97,7 @@ public:
 		return _reference_video;
 	}
 
-	bool can_reference_video (std::list<std::string> &) const;
+	bool can_reference_video (std::string &) const;
 
 	void set_reference_audio (bool r);
 
@@ -106,7 +106,7 @@ public:
 		return _reference_audio;
 	}
 
-	bool can_reference_audio (std::list<std::string> &) const;
+	bool can_reference_audio (std::string &) const;
 
 	void set_reference_subtitle (bool r);
 
@@ -115,7 +115,7 @@ public:
 		return _reference_subtitle;
 	}
 
-	bool can_reference_subtitle (std::list<std::string> &) const;
+	bool can_reference_subtitle (std::string &) const;
 
 	void set_cpl (std::string id);
 
@@ -139,7 +139,7 @@ private:
 	bool can_reference (
 		boost::function <boost::shared_ptr<ContentPart> (boost::shared_ptr<const Content>)>,
 		std::string overlapping,
-		std::list<std::string>& why_not
+		std::string& why_not
 		) const;
 
 	std::string _name;
-- 
cgit v1.2.3