From 5980cee25f537c54fc1155de5492bddd92bfa338 Mon Sep 17 00:00:00 2001 From: Carl Hetherington Date: Tue, 22 Aug 2023 23:45:13 +0200 Subject: Accept any case for subtitle alignment specifiers (#2602). SMPTE seems pretty clear to me that these should be lower case, but counterexamples have been seen in the wild and it seems to make sense to allow them. Perhaps the verifier should raise an error with non-lower-case examples. --- test/v_align_test.cc | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) (limited to 'test/v_align_test.cc') diff --git a/test/v_align_test.cc b/test/v_align_test.cc index 69951390..6bd77e44 100644 --- a/test/v_align_test.cc +++ b/test/v_align_test.cc @@ -42,8 +42,10 @@ BOOST_AUTO_TEST_CASE(v_align_test) BOOST_CHECK(dcp::string_to_valign("top") == dcp::VAlign::TOP); BOOST_CHECK(dcp::string_to_valign("center") == dcp::VAlign::CENTER); BOOST_CHECK(dcp::string_to_valign("bottom") == dcp::VAlign::BOTTOM); - BOOST_CHECK_THROW(dcp::string_to_valign("TOP"), dcp::ReadError); - BOOST_CHECK_THROW(dcp::string_to_valign("CENter"), dcp::ReadError); - BOOST_CHECK_THROW(dcp::string_to_valign("Right"), dcp::ReadError); + BOOST_CHECK(dcp::string_to_valign("TOP") == dcp::VAlign::TOP); + BOOST_CHECK(dcp::string_to_valign("CENter") == dcp::VAlign::CENTER); + BOOST_CHECK(dcp::string_to_valign("Bottom") == dcp::VAlign::BOTTOM); + + BOOST_CHECK_THROW(dcp::string_to_valign("ostrabagalous"), dcp::ReadError); } -- cgit v1.2.3