From: Carl Hetherington <cth@carlh.net>
Date: Sun, 10 Nov 2013 17:14:43 +0000 (+0000)
Subject: Hopefully get rid of spurious black lines around preview.
X-Git-Tag: v2.0.48~1167
X-Git-Url: https://main.carlh.net/gitweb/?p=dcpomatic.git;a=commitdiff_plain;h=3021ac636b39e61df5c90f9c3e75fde66d6fb97f

Hopefully get rid of spurious black lines around preview.
---

diff --git a/ChangeLog b/ChangeLog
index 851a85b52..c48bdb89b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2013-11-10  Carl Hetherington  <cth@carlh.net>
+
+	* Hopefully get rid of spurious black lines around
+	preview.
+
 2013-11-08  Carl Hetherington  <carl@ubuntu>
 
 	* Fix strange behaviour of J2K bandwidth control
diff --git a/src/lib/util.cc b/src/lib/util.cc
index e2ce94dd9..484c4fb9b 100644
--- a/src/lib/util.cc
+++ b/src/lib/util.cc
@@ -930,8 +930,8 @@ libdcp::Size
 fit_ratio_within (float ratio, libdcp::Size full_frame)
 {
 	if (ratio < full_frame.ratio ()) {
-		return libdcp::Size (full_frame.height * ratio, full_frame.height);
+		return libdcp::Size (rint (full_frame.height * ratio), full_frame.height);
 	}
 	
-	return libdcp::Size (full_frame.width, full_frame.width / ratio);
+	return libdcp::Size (full_frame.width, rint (full_frame.width / ratio));
 }