/*
- Copyright (C) 2014 Carl Hetherington <cth@carlh.net>
+ Copyright (C) 2014-2021 Carl Hetherington <cth@carlh.net>
- This program is free software; you can redistribute it and/or modify
+ This file is part of libdcp.
+
+ libdcp is free software; you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
the Free Software Foundation; either version 2 of the License, or
(at your option) any later version.
- This program is distributed in the hope that it will be useful,
+ libdcp is distributed in the hope that it will be useful,
but WITHOUT ANY WARRANTY; without even the implied warranty of
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
GNU General Public License for more details.
You should have received a copy of the GNU General Public License
- along with this program; if not, write to the Free Software
- Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+ along with libdcp. If not, see <http://www.gnu.org/licenses/>.
+
+ In addition, as a special exception, the copyright holders give
+ permission to link the code of portions of this program with the
+ OpenSSL library under certain conditions as described in each
+ individual source file, and distribute linked combinations
+ including the two.
+ You must obey the GNU General Public License in all respects
+ for all of the code used other than OpenSSL. If you modify
+ file(s) with this exception, you may extend this exception to your
+ version of the file(s), but you are not obligated to do so. If you
+ do not wish to do so, delete this exception statement from your
+ version. If you delete this exception statement from all source
+ files in the program, then also delete it here.
*/
+
#include "cpl.h"
+#include "mono_picture_asset.h"
#include "reel_mono_picture_asset.h"
-#include "mono_picture_mxf.h"
+#include "warnings.h"
#include <libcxml/cxml.h>
+LIBDCP_DISABLE_WARNINGS
+#include <libxml++/libxml++.h>
+LIBDCP_ENABLE_WARNINGS
#include <boost/test/unit_test.hpp>
-using boost::shared_ptr;
-/* Test for a reported bug where <ScreenAspectRatio> in Interop files uses
- excessive decimal places and (sometimes) the wrong decimal point character.
-*/
+using std::string;
+using std::shared_ptr;
+using std::make_shared;
+
+static void
+check (shared_ptr<dcp::ReelMonoPictureAsset> pa, dcp::Fraction frac, string sar)
+{
+ pa->set_screen_aspect_ratio (frac);
+ xmlpp::Document doc;
+ auto el = doc.create_root_node ("Test");
+ pa->write_to_cpl (el, dcp::Standard::INTEROP);
+
+ cxml::Node node (el);
+ BOOST_CHECK_EQUAL (node.node_child("MainPicture")->string_child ("ScreenAspectRatio"), sar);
+}
+
+
+/** Test for a reported bug where <ScreenAspectRatio> in Interop files uses
+ * excessive decimal places and (sometimes) the wrong decimal point character.
+ * Also check that we correctly use one of the allowed <ScreenAspectRatio>
+ * values with Interop.
+ */
BOOST_AUTO_TEST_CASE (cpl_sar)
{
- shared_ptr<dcp::ReelMonoPictureAsset> pa (
- new dcp::ReelMonoPictureAsset (
- shared_ptr<dcp::MonoPictureMXF> (new dcp::MonoPictureMXF ("test/ref/DCP/dcp_test1/video.mxf")),
- 0
- )
+ auto pa = make_shared<dcp::ReelMonoPictureAsset>(
+ make_shared<dcp::MonoPictureAsset>("test/ref/DCP/dcp_test1/video.mxf"), 0
);
- {
- pa->set_screen_aspect_ratio (dcp::Fraction (1998, 1080));
- xmlpp::Document doc;
- xmlpp::Element* el = doc.create_root_node ("Test");
- pa->write_to_cpl (el, dcp::INTEROP);
-
- cxml::Node node (el);
- BOOST_CHECK_EQUAL (node.node_child("MainPicture")->string_child ("ScreenAspectRatio"), "1.85");
- }
-
- {
- pa->set_screen_aspect_ratio (dcp::Fraction (2048, 858));
- xmlpp::Document doc;
- xmlpp::Element* el = doc.create_root_node ("Test");
- pa->write_to_cpl (el, dcp::INTEROP);
-
- cxml::Node node (el);
- BOOST_CHECK_EQUAL (node.node_child("MainPicture")->string_child ("ScreenAspectRatio"), "2.39");
- }
+ /* Easy ones */
+ check (pa, dcp::Fraction(1998, 1080), "1.85");
+ check (pa, dcp::Fraction(2048, 858), "2.39");
+
+ /* Check the use of the allowed values */
+
+ /* Just less then, equal to and just more than 1.33 */
+ check (pa, dcp::Fraction(1200, 1000), "1.33");
+ check (pa, dcp::Fraction(1330, 1000), "1.33");
+ check (pa, dcp::Fraction(1430, 1000), "1.33");
+
+ /* Same for 1.66 */
+ check (pa, dcp::Fraction(1600, 1000), "1.66");
+ check (pa, dcp::Fraction(1660, 1000), "1.66");
+ check (pa, dcp::Fraction(1670, 1000), "1.66");
+
+ /* 1.77 */
+ check (pa, dcp::Fraction(1750, 1000), "1.77");
+ check (pa, dcp::Fraction(1770, 1000), "1.77");
+ check (pa, dcp::Fraction(1800, 1000), "1.77");
+
+ /* 1.85 */
+ check (pa, dcp::Fraction(1820, 1000), "1.85");
+ check (pa, dcp::Fraction(1850, 1000), "1.85");
+ check (pa, dcp::Fraction(1910, 1000), "1.85");
+
+ /* 2.00 */
+ check (pa, dcp::Fraction(1999, 1000), "2.00");
+ check (pa, dcp::Fraction(2000, 1000), "2.00");
+ check (pa, dcp::Fraction(2001, 1000), "2.00");
+
+ /* 2.39 */
+ check (pa, dcp::Fraction(2350, 1000), "2.39");
+ check (pa, dcp::Fraction(2390, 1000), "2.39");
+ check (pa, dcp::Fraction(2500, 1000), "2.39");
}