Replace list with vector in most of the API.
[libsub.git] / test / dcp_reader_test.cc
index 5da9bf0b22ee13a38dc377c1bcad39d9a8003ccb..9105c283cb588e10e8fcc61a3d5a73a8a10a12b1 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2014 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2014-2015 Carl Hetherington <cth@carlh.net>
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
 #include "dcp_reader.h"
 #include "collect.h"
 #include <boost/test/unit_test.hpp>
-#include <fstream>
+#include <boost/optional/optional_io.hpp>
 
-using std::list;
-using std::cout;
-using std::ifstream;
-using boost::shared_ptr;
+using std::shared_ptr;
+using std::vector;
 
 /* Test reading of a DCP XML file */
 BOOST_AUTO_TEST_CASE (dcp_reader_test1)
 {
-       ifstream file ("test/data/test1.xml");
-       sub::DCPReader reader (file);
-       list<sub::Subtitle> subs = sub::collect<list<sub::Subtitle> > (reader.subtitles ());
+       sub::DCPReader reader ("test/data/test1.xml");
+       auto subs = sub::collect<vector<sub::Subtitle>> (reader.subtitles());
 
-       list<sub::Subtitle>::iterator i = subs.begin ();
-       BOOST_CHECK (i != subs.end ());
-       BOOST_CHECK_EQUAL (i->from.metric().get(), sub::MetricTime (0, 0, 5, 198 * 4));
-       BOOST_CHECK_EQUAL (i->to.metric(), sub::MetricTime (0, 0, 7, 115 * 4));
-       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::MetricTime (0, 0, 0, 4));
-       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::MetricTime (0, 0, 0, 4));
+       auto i = subs.begin();
+       BOOST_REQUIRE (i != subs.end());
+       BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 0, 5, 198 * 4));
+       BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 0, 7, 115 * 4));
+       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::Time::from_hms (0, 0, 0, 4));
+       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::Time::from_hms (0, 0, 0, 4));
 
        {
-               list<sub::Line>::iterator j = i->lines.begin ();
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
+               auto j = i->lines.begin();
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
                sub::Block b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               BOOST_CHECK_EQUAL (b.font.get(), "theFontId");
                BOOST_CHECK_EQUAL (b.italic, false);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (39) / (72 * 11));
@@ -57,22 +54,22 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test1)
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
 
                ++j;
-               BOOST_CHECK (j == i->lines.end ());
+               BOOST_REQUIRE (j == i->lines.end ());
        }
 
        ++i;
-       BOOST_CHECK (i != subs.end ());
-       BOOST_CHECK_EQUAL (i->from.metric().get(), sub::MetricTime (0, 0, 7, 177 * 4));
-       BOOST_CHECK_EQUAL (i->to.metric().get(), sub::MetricTime (0, 0, 11, 31 * 4));
-       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::MetricTime (0, 0, 0, 4));
-       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::MetricTime (0, 0, 0, 4));
+       BOOST_REQUIRE (i != subs.end ());
+       BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 0, 7, 177 * 4));
+       BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 0, 11, 31 * 4));
+       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::Time::from_hms (0, 0, 0, 4));
+       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::Time::from_hms (0, 0, 0, 4));
 
        {
-               list<sub::Line>::iterator j = i->lines.begin ();
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
-               sub::Block b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               auto j = i->lines.begin();
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
+               auto b = j->blocks.front ();
+               BOOST_CHECK_EQUAL (b.font.get(), "theFontId");
                BOOST_CHECK_EQUAL (b.italic, true);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (39) / (72 * 11));
@@ -83,10 +80,10 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test1)
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
 
                ++j;
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
                b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               BOOST_CHECK_EQUAL (b.font.get(), "theFontId");
                BOOST_CHECK_EQUAL (b.italic, false);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (39) / (72 * 11));
@@ -97,22 +94,22 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test1)
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
 
                ++j;
-               BOOST_CHECK (j == i->lines.end ());
+               BOOST_REQUIRE (j == i->lines.end ());
        }
 
        ++i;
-       BOOST_CHECK (i != subs.end ());
-       BOOST_CHECK_EQUAL (i->from.metric().get(), sub::MetricTime (0, 0, 11, 94 * 4));
-       BOOST_CHECK_EQUAL (i->to.metric().get(), sub::MetricTime (0, 0, 13, 63 * 4));
-       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::MetricTime (0, 0, 0, 4));
-       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::MetricTime (0, 0, 0, 4));
+       BOOST_REQUIRE (i != subs.end ());
+       BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 0, 11, 94 * 4));
+       BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 0, 13, 63 * 4));
+       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::Time::from_hms (0, 0, 0, 4));
+       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::Time::from_hms (0, 0, 0, 4));
 
        {
-               list<sub::Line>::iterator j = i->lines.begin ();
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
-               sub::Block b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               auto j = i->lines.begin();
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
+               auto b = j->blocks.front();
+               BOOST_CHECK_EQUAL (b.font.get(), "theFontId");
                BOOST_CHECK_EQUAL (b.italic, false);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (39) / (72 * 11));
@@ -123,22 +120,22 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test1)
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
 
                ++j;
-               BOOST_CHECK (j == i->lines.end ());
+               BOOST_REQUIRE (j == i->lines.end ());
        }
 
        ++i;
-       BOOST_CHECK (i != subs.end ());
-       BOOST_CHECK_EQUAL (i->from.metric(), sub::MetricTime (0, 0, 13, 104 * 4));
-       BOOST_CHECK_EQUAL (i->to.metric().get(), sub::MetricTime (0, 0, 15, 177 * 4));
-       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::MetricTime (0, 0, 0, 4));
-       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::MetricTime (0, 0, 0, 4));
+       BOOST_REQUIRE (i != subs.end ());
+       BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 0, 13, 104 * 4));
+       BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 0, 15, 177 * 4));
+       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::Time::from_hms (0, 0, 0, 4));
+       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::Time::from_hms (0, 0, 0, 4));
 
        {
-               list<sub::Line>::iterator j = i->lines.begin ();
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
-               sub::Block b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               auto j = i->lines.begin();
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
+               auto b = j->blocks.front();
+               BOOST_CHECK_EQUAL (b.font.get(), "theFontId");
                BOOST_CHECK_EQUAL (b.italic, false);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (39) / (72 * 11));
@@ -149,30 +146,29 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test1)
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
 
                ++j;
-               BOOST_CHECK (j == i->lines.end ());
+               BOOST_REQUIRE (j == i->lines.end ());
        }
 }
 
 /* And another one */
 BOOST_AUTO_TEST_CASE (dcp_reader_test2)
 {
-       ifstream file ("test/data/test2.xml");
-       sub::DCPReader reader (file);
-       list<sub::Subtitle> subs = sub::collect<list<sub::Subtitle> > (reader.subtitles ());
+       sub::DCPReader reader ("test/data/test2.xml");
+       auto subs = sub::collect<vector<sub::Subtitle>> (reader.subtitles ());
 
-       list<sub::Subtitle>::iterator i = subs.begin ();
-       BOOST_CHECK (i != subs.end ());
-       BOOST_CHECK_EQUAL (i->from.metric(), sub::MetricTime (0, 0, 41, 62 * 4));
-       BOOST_CHECK_EQUAL (i->to.metric().get(), sub::MetricTime (0, 0, 43, 52 * 4));
-       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::MetricTime (0, 0, 0, 0));
-       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::MetricTime (0, 0, 0, 0));
+       auto i = subs.begin ();
+       BOOST_REQUIRE (i != subs.end ());
+       BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 0, 41, 62 * 4));
+       BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 0, 43, 52 * 4));
+       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::Time::from_hms (0, 0, 0, 0));
+       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::Time::from_hms (0, 0, 0, 0));
 
        {
-               list<sub::Line>::iterator j = i->lines.begin ();
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
-               sub::Block b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               auto j = i->lines.begin ();
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
+               auto b = j->blocks.front ();
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, true);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -181,12 +177,12 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
                BOOST_CHECK_EQUAL (b.text, "At afternoon tea with John Peel");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
-               
+
                ++j;
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
                b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, true);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -199,20 +195,20 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
                ++j;
                BOOST_CHECK (j == i->lines.end ());
        }
-               
+
        ++i;
-       BOOST_CHECK (i != subs.end ());
-       BOOST_CHECK_EQUAL (i->from.metric(), sub::MetricTime (0, 0, 50, 42 * 4));
-       BOOST_CHECK_EQUAL (i->to.metric().get(), sub::MetricTime (0, 0, 52, 21 * 4));
-       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::MetricTime (0, 0, 0, 0));
-       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::MetricTime (0, 0, 0, 0));
+       BOOST_REQUIRE (i != subs.end ());
+       BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 0, 50, 42 * 4));
+       BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 0, 52, 21 * 4));
+       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::Time::from_hms (0, 0, 0, 0));
+       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::Time::from_hms (0, 0, 0, 0));
 
        {
-               list<sub::Line>::iterator j = i->lines.begin ();
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
-               sub::Block b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               auto j = i->lines.begin ();
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
+               auto b = j->blocks.front ();
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, true);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -223,10 +219,10 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
 
                ++j;
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
                b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, true);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -241,18 +237,18 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
        }
 
        ++i;
-       BOOST_CHECK (i != subs.end ());
-       BOOST_CHECK_EQUAL (i->from.metric().get(), sub::MetricTime (0, 1, 2, 208 * 4));
-       BOOST_CHECK_EQUAL (i->to.metric().get(), sub::MetricTime (0, 1, 04, 10 * 4));
-       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::MetricTime (0, 0, 0, 0));
-       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::MetricTime (0, 0, 0, 0));
+       BOOST_REQUIRE (i != subs.end ());
+       BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 1, 2, 208 * 4));
+       BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 1, 04, 10 * 4));
+       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::Time::from_hms (0, 0, 0, 0));
+       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::Time::from_hms (0, 0, 0, 0));
 
        {
-               list<sub::Line>::iterator j = i->lines.begin ();
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
-               sub::Block b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               auto j = i->lines.begin();
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
+               auto b = j->blocks.front();
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, true);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -261,42 +257,42 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
                BOOST_CHECK_EQUAL (b.text, "At home it depends how I feel.\"");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
-               
+
                ++j;
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
                b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, true);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
-               BOOST_CHECK_EQUAL (i->from.metric().get(), sub::MetricTime (0, 1, 2, 208 * 4));
-               BOOST_CHECK_EQUAL (i->to.metric().get(), sub::MetricTime (0, 1, 4, 10 * 4));
+               BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 1, 2, 208 * 4));
+               BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 1, 4, 10 * 4));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.95, 1);
                BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                BOOST_CHECK_EQUAL (b.text, "I spent a long weekend in Brighton");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
-               BOOST_CHECK_EQUAL (i->fade_up.get(), sub::MetricTime (0, 0, 0, 0));
-               BOOST_CHECK_EQUAL (i->fade_down.get(), sub::MetricTime (0, 0, 0, 0));
+               BOOST_CHECK_EQUAL (i->fade_up.get(), sub::Time::from_hms (0, 0, 0, 0));
+               BOOST_CHECK_EQUAL (i->fade_down.get(), sub::Time::from_hms (0, 0, 0, 0));
 
                ++j;
                BOOST_CHECK (j == i->lines.end ());
        }
 
        ++i;
-       BOOST_CHECK (i != subs.end ());
-       BOOST_CHECK_EQUAL (i->from.metric().get(), sub::MetricTime (0, 1, 15, 42 * 4));
-       BOOST_CHECK_EQUAL (i->to.metric().get(), sub::MetricTime (0, 1, 16, 42 * 4));
-       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::MetricTime (0, 0, 0, 0));
-       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::MetricTime (0, 0, 0, 0));
-       
+       BOOST_REQUIRE (i != subs.end ());
+       BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 1, 15, 42 * 4));
+       BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 1, 16, 42 * 4));
+       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::Time::from_hms (0, 0, 0, 0));
+       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::Time::from_hms (0, 0, 0, 0));
+
        {
-               list<sub::Line>::iterator j = i->lines.begin ();
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
-               sub::Block b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               auto j = i->lines.begin();
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
+               auto b = j->blocks.front();
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, true);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -305,12 +301,12 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
                BOOST_CHECK_EQUAL (b.text, "With the legendary Miss Enid Blyton");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
-               
+
                ++j;
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
                b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, true);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -325,18 +321,18 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
        }
 
        ++i;
-       BOOST_CHECK (i != subs.end ());
-       BOOST_CHECK_EQUAL (i->from.metric().get(), sub::MetricTime (0, 1, 20, 219 * 4));
-       BOOST_CHECK_EQUAL (i->to.metric().get(), sub::MetricTime (0, 1, 22, 73 * 4));
-       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::MetricTime (0, 0, 0, 0));
-       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::MetricTime (0, 0, 0, 0));   
-       
+       BOOST_REQUIRE (i != subs.end ());
+       BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 1, 20, 219 * 4));
+       BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 1, 22, 73 * 4));
+       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::Time::from_hms (0, 0, 0, 0));
+       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::Time::from_hms (0, 0, 0, 0));
+
        {
-               list<sub::Line>::iterator j = i->lines.begin ();
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
-               sub::Block b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               auto j = i->lines.begin();
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
+               auto b = j->blocks.front();
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, true);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -345,12 +341,12 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
                BOOST_CHECK_EQUAL (b.text, "and I'll show you my body\"");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
-               
+
                ++j;
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
                b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, true);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -365,18 +361,18 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
        }
 
        ++i;
-       BOOST_CHECK (i != subs.end ());
-       BOOST_CHECK_EQUAL (i->from.metric().get(), sub::MetricTime (0, 1, 27, 115 * 4));
-       BOOST_CHECK_EQUAL (i->to.metric().get(), sub::MetricTime (0, 1, 28, 208 * 4));
-       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::MetricTime (0, 0, 0, 0));
-       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::MetricTime (0, 0, 0, 0));
-       
+       BOOST_REQUIRE (i != subs.end ());
+       BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 1, 27, 115 * 4));
+       BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 1, 28, 208 * 4));
+       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::Time::from_hms (0, 0, 0, 0));
+       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::Time::from_hms (0, 0, 0, 0));
+
        {
-               list<sub::Line>::iterator j = i->lines.begin ();
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
-               sub::Block b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               auto j = i->lines.begin();
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
+               auto b = j->blocks.front();
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, true);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -385,12 +381,12 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
                BOOST_CHECK_EQUAL (b.text, "That curious creature the Sphinx");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
-               
+
                ++j;
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
                b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, true);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -405,18 +401,18 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
        }
 
        ++i;
-       BOOST_CHECK (i != subs.end ());
-       BOOST_CHECK_EQUAL (i->from.metric().get(), sub::MetricTime (0, 1, 42, 229 * 4));
-       BOOST_CHECK_EQUAL (i->to.metric().get(), sub::MetricTime (0, 1, 45, 62 * 4));
-       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::MetricTime (0, 0, 0, 0));
-       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::MetricTime (0, 0, 0, 0));
+       BOOST_REQUIRE (i != subs.end ());
+       BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 1, 42, 229 * 4));
+       BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 1, 45, 62 * 4));
+       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::Time::from_hms (0, 0, 0, 0));
+       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::Time::from_hms (0, 0, 0, 0));
 
        {
-               list<sub::Line>::iterator j = i->lines.begin ();
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
-               sub::Block b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               auto j = i->lines.begin();
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
+               auto b = j->blocks.front();
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, false);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -425,12 +421,12 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
                BOOST_CHECK_EQUAL (b.text, "It sits there and smirks");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
-               
+
                ++j;
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
                b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, false);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -445,18 +441,18 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
        }
 
        ++i;
-       BOOST_CHECK (i != subs.end ());
-       BOOST_CHECK_EQUAL (i->from.metric().get(), sub::MetricTime (0, 1, 45, 146 * 4));
-       BOOST_CHECK_EQUAL (i->to.metric().get(), sub::MetricTime (0, 1, 47, 94 * 4));
-       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::MetricTime (0, 0, 0, 0));
-       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::MetricTime (0, 0, 0, 0));
-       
+       BOOST_REQUIRE (i != subs.end ());
+       BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 1, 45, 146 * 4));
+       BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 1, 47, 94 * 4));
+       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::Time::from_hms (0, 0, 0, 0));
+       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::Time::from_hms (0, 0, 0, 0));
+
        {
-               list<sub::Line>::iterator j = i->lines.begin ();
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
-               sub::Block b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               auto j = i->lines.begin();
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
+               auto b = j->blocks.front();
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, false);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -465,12 +461,12 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
                BOOST_CHECK_EQUAL (b.text, "Then when you're not looking, it winks.");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
-               
+
                ++j;
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
                b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, false);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -485,18 +481,18 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
        }
 
        ++i;
-       BOOST_CHECK (i != subs.end ());
-       BOOST_CHECK_EQUAL (i->from.metric().get(), sub::MetricTime (0, 1, 47, 146 * 4));
-       BOOST_CHECK_EQUAL (i->to.metric().get(), sub::MetricTime (0, 1, 48, 167 * 4));
-       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::MetricTime (0, 0, 0, 0));
-       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::MetricTime (0, 0, 0, 0));
-       
+       BOOST_REQUIRE (i != subs.end ());
+       BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 1, 47, 146 * 4));
+       BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 1, 48, 167 * 4));
+       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::Time::from_hms (0, 0, 0, 0));
+       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::Time::from_hms (0, 0, 0, 0));
+
        {
-               list<sub::Line>::iterator j = i->lines.begin ();
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
-               sub::Block b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               auto j = i->lines.begin();
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
+               auto b = j->blocks.front();
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, false);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -505,12 +501,12 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
                BOOST_CHECK_EQUAL (b.text, "Out mooning, at night, on the ledge");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
-               
+
                ++j;
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
                b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, false);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -523,20 +519,20 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
                ++j;
                BOOST_CHECK (j == i->lines.end ());
        }
-               
+
        ++i;
-       BOOST_CHECK (i != subs.end ());
-       BOOST_CHECK_EQUAL (i->from.metric().get(), sub::MetricTime (0, 1, 53, 21 * 4));
-       BOOST_CHECK_EQUAL (i->to.metric().get(), sub::MetricTime (0, 1, 56, 10 * 4));
-       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::MetricTime (0, 0, 0, 0));
-       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::MetricTime (0, 0, 0, 0));
-       
+       BOOST_REQUIRE (i != subs.end ());
+       BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 1, 53, 21 * 4));
+       BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 1, 56, 10 * 4));
+       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::Time::from_hms (0, 0, 0, 0));
+       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::Time::from_hms (0, 0, 0, 0));
+
        {
-               list<sub::Line>::iterator j = i->lines.begin ();
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
-               sub::Block b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               auto j = i->lines.begin();
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
+               auto b = j->blocks.front();
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, false);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -545,12 +541,12 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
                BOOST_CHECK_EQUAL (b.text, "Is the maestro, James Brown");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
-               
+
                ++j;
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
                b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, false);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -563,20 +559,20 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
                ++j;
                BOOST_CHECK (j == i->lines.end ());
        }
-       
+
        ++i;
-       BOOST_CHECK (i != subs.end ());
-       BOOST_CHECK_EQUAL (i->from.metric().get(), sub::MetricTime (0, 2, 5, 208 * 4));
-       BOOST_CHECK_EQUAL (i->to.metric().get(), sub::MetricTime (0, 2, 7, 31 * 4));
-       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::MetricTime (0, 0, 0, 0));
-       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::MetricTime (0, 0, 0, 0));
-       
+       BOOST_REQUIRE (i != subs.end ());
+       BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 2, 5, 208 * 4));
+       BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 2, 7, 31 * 4));
+       BOOST_CHECK_EQUAL (i->fade_up.get(), sub::Time::from_hms (0, 0, 0, 0));
+       BOOST_CHECK_EQUAL (i->fade_down.get(), sub::Time::from_hms (0, 0, 0, 0));
+
        {
-               list<sub::Line>::iterator j = i->lines.begin ();
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
-               sub::Block b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               auto j = i->lines.begin();
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
+               auto b = j->blocks.front();
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, true);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));
@@ -585,12 +581,12 @@ BOOST_AUTO_TEST_CASE (dcp_reader_test2)
                BOOST_CHECK_EQUAL (b.text, "HELLO");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
-               
+
                ++j;
-               BOOST_CHECK (j != i->lines.end ());
-               BOOST_CHECK_EQUAL (j->blocks.size(), 1);
+               BOOST_REQUIRE (j != i->lines.end ());
+               BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
                b = j->blocks.front ();
-               BOOST_CHECK_EQUAL (b.font, "Arial");
+               BOOST_CHECK_EQUAL (b.font.get(), "theFont");
                BOOST_CHECK_EQUAL (b.italic, true);
                BOOST_CHECK (b.colour == sub::Colour (1, 1, 1));
                BOOST_CHECK_EQUAL (b.font_size.proportional().get(), float (42) / (72 * 11));