Allow any ratio to appear in the ISDCF name as an interior aspect ratio (#2030).

1 files changed, 7 insertions, 11 deletions

diff --git a/src/lib/film.cc b/src/lib/film.cc
index 0b717738d..62cbf0e50 100644
--- a/src/lib/film.cc
+++ b/src/lib/film.cc
@@ -911,19 +911,15 @@ Film::isdcf_name (bool if_created_now) const
 
 	/* Interior aspect ratio.  The standard says we don't do this for trailers, for some strange reason */
 	if (dcp_content_type() && dcp_content_type()->libdcp_kind() != dcp::ContentKind::TRAILER) {
-		Ratio const* content_ratio = nullptr;
-		for (auto i: content ()) {
-			if (i->video) {
-				/* Here's the first piece of video content */
-				content_ratio = Ratio::nearest_from_ratio(i->video->scaled_size(frame_size()).ratio());
-				break;
+		auto cl = content();
+		auto first_video = std::find_if(cl.begin(), cl.end(), [](shared_ptr<Content> c) { return static_cast<bool>(c->video); });
+		if (first_video != cl.end()) {
+			auto first_ratio = lrintf((*first_video)->video->scaled_size(frame_size()).ratio() * 100);
+			auto container_ratio = lrintf(container()->ratio() * 100);
+			if (first_ratio != container_ratio) {
+				d += "-" + dcp::raw_convert<string>(first_ratio);
 			}
 		}
-
-		if (content_ratio && content_ratio != container()) {
-			/* This needs to be the numeric version of the ratio, and ::id() is close enough */
-			d += "-" + content_ratio->id();
-		}
 	}
 
 	auto audio_langs = audio_languages();