Bump libdcp for bug fix.
[libsub.git] / test / dcp_reader_test.cc
index c73207e09ee461e2ba0a93a4f755ce66fee98133..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 (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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.15, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::BOTTOM);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::BOTTOM_OF_SCREEN);
                BOOST_CHECK_EQUAL (b.text, "My jacket was Idi Amin's");
                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_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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.21, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::BOTTOM);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::BOTTOM_OF_SCREEN);
                BOOST_CHECK_EQUAL (b.text, "My corset was H.M. The Queen's");
                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(), "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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.15, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::BOTTOM);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::BOTTOM_OF_SCREEN);
                BOOST_CHECK_EQUAL (b.text, "My large wonderbra");
                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_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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.15, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::BOTTOM);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::BOTTOM_OF_SCREEN);
                BOOST_CHECK_EQUAL (b.text, "Once belonged to the Shah");
                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_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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.15, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::BOTTOM);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::BOTTOM_OF_SCREEN);
                BOOST_CHECK_EQUAL (b.text, "And these are Roy Hattersley's jeans");
                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_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 = collect (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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.89, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.95, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                BOOST_CHECK_EQUAL (b.text, "I enquired if his accent was real");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
@@ -199,39 +195,39 @@ 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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.89, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                BOOST_CHECK_EQUAL (b.text, "He said \"out of the house");
                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_CLOSE (j->vertical_position.proportional.get(), 0.95, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                BOOST_CHECK_EQUAL (b.text, "I'm incredibly scouse");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
@@ -241,81 +237,81 @@ 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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.89, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                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);
+               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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.89, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.95, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                BOOST_CHECK_EQUAL (b.text, "She said \"you be Noddy");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
@@ -325,37 +321,37 @@ 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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.89, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.95, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                BOOST_CHECK_EQUAL (b.text, "But Big Ears kept turning the light on.");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
@@ -365,37 +361,37 @@ 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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.89, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.95, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                BOOST_CHECK_EQUAL (b.text, "Is smarter than anyone thinks");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
@@ -405,37 +401,37 @@ 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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.89, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.95, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                BOOST_CHECK_EQUAL (b.text, "And you don't think it works");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
@@ -445,37 +441,37 @@ 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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.89, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.95, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                BOOST_CHECK_EQUAL (b.text, "When it snows you will find Sister Sledge");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
@@ -485,37 +481,37 @@ 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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.89, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.95, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                BOOST_CHECK_EQUAL (b.text, "One storey down");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
@@ -523,39 +519,39 @@ 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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.89, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.95, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                BOOST_CHECK_EQUAL (b.text, "Displaying his meat and two veg.");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));
@@ -563,39 +559,39 @@ 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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.89, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                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));
                BOOST_CHECK_CLOSE (j->vertical_position.proportional.get(), 0.95, 1);
-               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP);
+               BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::TOP_OF_SCREEN);
                BOOST_CHECK_EQUAL (b.text, "WORLD");
                BOOST_CHECK_EQUAL (b.effect, sub::BORDER);
                BOOST_CHECK (b.effect_colour.get() == sub::Colour (0, 0, 0));