summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCarl Hetherington <cth@carlh.net>2013-04-10 19:52:20 +0100
committerCarl Hetherington <cth@carlh.net>2013-04-10 19:52:20 +0100
commitbb1251a3b0abe2343962e126b690d97ebf8394c5 (patch)
treeca816feb4cf1133d0eab20b90d90c4f9bef0908b
parent9bdd8cc51942a13e360dde4efc04b3ca417c8b94 (diff)
Fix the merge.
-rw-r--r--src/lib/format.cc2
-rw-r--r--test/test.cc26
2 files changed, 1 insertions, 27 deletions
diff --git a/src/lib/format.cc b/src/lib/format.cc
index cce8762bd..3d18edb3e 100644
--- a/src/lib/format.cc
+++ b/src/lib/format.cc
@@ -200,7 +200,7 @@ FixedFormat::FixedFormat (int r, libdcp::Size dcp, string id, string n, string d
int
Format::dcp_padding (shared_ptr<const Film> f) const
{
- int p = rint ((_dcp_size.width - (_dcp_size.height * ratio(f))) / 2.0);
+ int p = rint ((_dcp_size.width - (_dcp_size.height * ratio_as_float(f))) / 2.0);
/* This comes out -ve for Scope; bodge it */
if (p < 0) {
diff --git a/test/test.cc b/test/test.cc
index 4d25d50f8..592bad836 100644
--- a/test/test.cc
+++ b/test/test.cc
@@ -197,32 +197,6 @@ BOOST_AUTO_TEST_CASE (format_test)
// BOOST_CHECK_EQUAL (f->ratio_as_integer(shared_ptr<const Film> ()), 239);
}
-/* Test VariableFormat-based scaling of content */
-BOOST_AUTO_TEST_CASE (scaling_test)
-{
- shared_ptr<Film> film (new Film (test_film_dir ("scaling_test").string(), false));
-
- /* 4:3 ratio */
- film->set_size (libdcp::Size (320, 240));
-
- /* This format should preserve aspect ratio of the source */
- Format const * format = Format::from_id ("var-185");
-
- /* We should have enough padding that the result is 4:3,
- which would be 1440 pixels.
- */
- BOOST_CHECK_EQUAL (format->dcp_padding (film), (1998 - 1440) / 2);
-
- /* This crops it to 1.291666667 */
- film->set_left_crop (5);
- film->set_right_crop (5);
-
- /* We should now have enough padding that the result is 1.29166667,
- which would be 1395 pixels.
- */
- BOOST_CHECK_EQUAL (format->dcp_padding (film), rint ((1998 - 1395) / 2.0));
-}
-
BOOST_AUTO_TEST_CASE (util_test)
{
string t = "Hello this is a string \"with quotes\" and indeed without them";