diff options
| author | Carl Hetherington <cth@carlh.net> | 2013-04-10 19:52:20 +0100 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2013-04-10 19:52:20 +0100 |
| commit | bb1251a3b0abe2343962e126b690d97ebf8394c5 (patch) | |
| tree | ca816feb4cf1133d0eab20b90d90c4f9bef0908b | |
| parent | 9bdd8cc51942a13e360dde4efc04b3ca417c8b94 (diff) | |
Fix the merge.
| -rw-r--r-- | src/lib/format.cc | 2 | ||||
| -rw-r--r-- | test/test.cc | 26 |
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"; |
