c++11 tidying.

[libsub.git] / src / dcp_reader.cc

diff --git a/src/dcp_reader.cc b/src/dcp_reader.cc
index 5144a4157b8b149e307b096e85badc7ac222505c..82808b3c1336a8c43d3fe337ef896bff857b6a18 100644 (file)
--- a/src/dcp_reader.cc
+++ b/src/dcp_reader.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2014-2017 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
     it under the terms of the GNU General Public License as published by
@@ -23,15 +23,15 @@
 #include <dcp/subtitle_string.h>
 #include <dcp/interop_subtitle_asset.h>
 #include <dcp/smpte_subtitle_asset.h>
-#include <boost/foreach.hpp>
 #include <boost/filesystem.hpp>
 
 using std::list;
 using std::cout;
 using std::string;
 using std::exception;
-using boost::shared_ptr;
+using std::shared_ptr;
 using boost::optional;
+using std::dynamic_pointer_cast;
 using namespace sub;
 
 static Time
@@ -71,13 +71,20 @@ DCPReader::DCPReader (boost::filesystem::path file)
        }
 
 
-       BOOST_FOREACH (dcp::SubtitleString const & i, sc->subtitles ()) {
+       for (auto i: sc->subtitles()) {
+
+               /* We don't deal with image subs */
+               auto is = dynamic_pointer_cast<dcp::SubtitleString>(i);
+               if (!is) {
+                       continue;
+               }
+
                RawSubtitle rs;
-               rs.text = i.text ();
-               rs.font = i.font ();
-               rs.font_size = FontSize::from_proportional (i.size() / (72.0 * 11.0));
+               rs.text = is->text ();
+               rs.font = is->font ();
+               rs.font_size = FontSize::from_proportional (is->size() / (72.0 * 11.0));
 
-               switch (i.effect ()) {
+               switch (is->effect ()) {
                case dcp::BORDER:
                        rs.effect = BORDER;
                        break;
@@ -88,43 +95,43 @@ DCPReader::DCPReader (boost::filesystem::path file)
                        break;
                }
 
-               rs.effect_colour = dcp_to_sub_colour (i.effect_colour());
+               rs.effect_colour = dcp_to_sub_colour (is->effect_colour());
 
-               rs.colour = dcp_to_sub_colour (i.colour());
-               rs.bold = i.bold ();
-               rs.italic = i.italic ();
-               rs.underline = i.underline ();
+               rs.colour = dcp_to_sub_colour (is->colour());
+               rs.bold = is->bold ();
+               rs.italic = is->italic ();
+               rs.underline = is->underline ();
 
-               switch (i.h_align()) {
+               switch (is->h_align()) {
                case dcp::HALIGN_LEFT:
-                       rs.horizontal_position = LEFT;
+                       rs.horizontal_position.reference = LEFT_OF_SCREEN;
                        break;
                case dcp::HALIGN_CENTER:
-                       rs.horizontal_position = CENTRE;
+                       rs.horizontal_position.reference = HORIZONTAL_CENTRE_OF_SCREEN;
                        break;
                case dcp::HALIGN_RIGHT:
-                       rs.horizontal_position = RIGHT;
+                       rs.horizontal_position.reference = RIGHT_OF_SCREEN;
                        break;
                }
 
-               rs.vertical_position.proportional = i.v_position();
-               switch (i.v_align()) {
+               rs.vertical_position.proportional = is->v_position();
+               switch (is->v_align()) {
                case dcp::VALIGN_TOP:
                        rs.vertical_position.reference = TOP_OF_SCREEN;
                        break;
                case dcp::VALIGN_CENTER:
-                       rs.vertical_position.reference = CENTRE_OF_SCREEN;
+                       rs.vertical_position.reference = VERTICAL_CENTRE_OF_SCREEN;
                        break;
                case dcp::VALIGN_BOTTOM:
                        rs.vertical_position.reference = BOTTOM_OF_SCREEN;
                        break;
                }
 
-               rs.from = dcp_to_sub_time (i.in ());
-               rs.to = dcp_to_sub_time (i.out ());
+               rs.from = dcp_to_sub_time (is->in ());
+               rs.to = dcp_to_sub_time (is->out ());
 
-               rs.fade_up = dcp_to_sub_time (i.fade_up_time ());
-               rs.fade_down = dcp_to_sub_time (i.fade_down_time ());
+               rs.fade_up = dcp_to_sub_time (is->fade_up_time ());
+               rs.fade_down = dcp_to_sub_time (is->fade_down_time ());
 
                _subs.push_back (rs);
        }