More renaming.
authorCarl Hetherington <cth@carlh.net>
Thu, 19 Jul 2018 00:09:19 +0000 (01:09 +0100)
committerCarl Hetherington <cth@carlh.net>
Thu, 19 Jul 2018 22:45:19 +0000 (23:45 +0100)
15 files changed:
src/lib/content_factory.cc
src/lib/decoder_factory.cc
src/lib/plain_text.cc
src/lib/plain_text.h
src/lib/plain_text_content.cc
src/lib/plain_text_content.h
src/lib/plain_text_decoder.cc
src/lib/plain_text_decoder.h
src/wx/content_panel.cc
test/ffmpeg_encoder_test.cc
test/player_test.cc
test/reels_test.cc
test/srt_subtitle_test.cc
test/ssa_subtitle_test.cc
test/subtitle_reel_number_test.cc

index 24ad955..4e8880f 100644 (file)
@@ -88,7 +88,7 @@ content_factory (shared_ptr<const Film> film, cxml::NodePtr node, int version, l
                        );
 
        } else if (type == "SubRip" || type == "TextSubtitle") {
-               content.reset (new PlainText (film, node, version));
+               content.reset (new PlainTextContent (film, node, version));
        } else if (type == "DCP") {
                content.reset (new DCPContent (film, node, version));
        } else if (type == "DCPSubtitle") {
@@ -210,7 +210,7 @@ content_factory (shared_ptr<const Film> film, boost::filesystem::path path)
                if (valid_image_file (path)) {
                        single.reset (new ImageContent (film, path));
                } else if (ext == ".srt" || ext == ".ssa" || ext == ".ass") {
-                       single.reset (new PlainText (film, path));
+                       single.reset (new PlainTextContent (film, path));
                } else if (ext == ".xml") {
                        cxml::Document doc;
                        doc.read_file (path);
index 4301edd..73a5b66 100644 (file)
@@ -54,9 +54,9 @@ decoder_factory (shared_ptr<const Content> content, shared_ptr<Log> log, bool fa
                return shared_ptr<Decoder> (new ImageDecoder (ic, log));
        }
 
-       shared_ptr<const PlainText> rc = dynamic_pointer_cast<const PlainText> (content);
+       shared_ptr<const PlainTextContent> rc = dynamic_pointer_cast<const PlainTextContent> (content);
        if (rc) {
-               return shared_ptr<Decoder> (new TextTextDecoder (rc, log));
+               return shared_ptr<Decoder> (new PlainTextDecoder (rc, log));
        }
 
        shared_ptr<const DCPTextContent> dsc = dynamic_pointer_cast<const DCPTextContent> (content);
index ee9bc4b..58035a0 100644 (file)
@@ -39,7 +39,7 @@ using boost::scoped_array;
 using boost::optional;
 using dcp::Data;
 
-TextSubtitle::TextSubtitle (shared_ptr<const PlainText> content)
+PlainText::PlainText (shared_ptr<const PlainTextContent> content)
 {
        Data in (content->path (0));
 
@@ -96,7 +96,7 @@ TextSubtitle::TextSubtitle (shared_ptr<const PlainText> content)
 
 /** @return time of first subtitle, if there is one */
 optional<ContentTime>
-TextSubtitle::first () const
+PlainText::first () const
 {
        if (_subtitles.empty()) {
                return optional<ContentTime>();
@@ -106,7 +106,7 @@ TextSubtitle::first () const
 }
 
 ContentTime
-TextSubtitle::length () const
+PlainText::length () const
 {
        if (_subtitles.empty ()) {
                return ContentTime ();
index 61d7147..01d3d77 100644 (file)
 #include <boost/shared_ptr.hpp>
 #include <vector>
 
-class PlainText;
+class PlainTextContent;
 class plain_text_time_test;
 class plain_text_coordinate_test;
 class plain_text_content_test;
 class plain_text_parse_test;
 
-class TextSubtitle
+class PlainText
 {
 public:
-       explicit TextSubtitle (boost::shared_ptr<const PlainText>);
+       explicit PlainText (boost::shared_ptr<const PlainTextContent>);
 
        boost::optional<ContentTime> first () const;
        ContentTime length () const;
index f51e365..a07dcb6 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2014-2016 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2014-2018 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -35,13 +35,13 @@ using std::cout;
 using boost::shared_ptr;
 using dcp::raw_convert;
 
-PlainText::PlainText (shared_ptr<const Film> film, boost::filesystem::path path)
+PlainTextContent::PlainTextContent (shared_ptr<const Film> film, boost::filesystem::path path)
        : Content (film, path)
 {
        subtitle.reset (new TextContent (this));
 }
 
-PlainText::PlainText (shared_ptr<const Film> film, cxml::ConstNodePtr node, int version)
+PlainTextContent::PlainTextContent (shared_ptr<const Film> film, cxml::ConstNodePtr node, int version)
        : Content (film, node)
        , _length (node->number_child<ContentTime::Type> ("Length"))
 {
@@ -49,10 +49,10 @@ PlainText::PlainText (shared_ptr<const Film> film, cxml::ConstNodePtr node, int
 }
 
 void
-PlainText::examine (boost::shared_ptr<Job> job)
+PlainTextContent::examine (boost::shared_ptr<Job> job)
 {
        Content::examine (job);
-       TextSubtitle s (shared_from_this ());
+       PlainText s (shared_from_this ());
 
        /* Default to turning these subtitles on */
        subtitle->set_use (true);
@@ -63,19 +63,19 @@ PlainText::examine (boost::shared_ptr<Job> job)
 }
 
 string
-PlainText::summary () const
+PlainTextContent::summary () const
 {
        return path_summary() + " " + _("[subtitles]");
 }
 
 string
-PlainText::technical_summary () const
+PlainTextContent::technical_summary () const
 {
        return Content::technical_summary() + " - " + _("Text subtitles");
 }
 
 void
-PlainText::as_xml (xmlpp::Node* node, bool with_paths) const
+PlainTextContent::as_xml (xmlpp::Node* node, bool with_paths) const
 {
        node->add_child("Type")->add_child_text ("TextSubtitle");
        Content::as_xml (node, with_paths);
@@ -88,7 +88,7 @@ PlainText::as_xml (xmlpp::Node* node, bool with_paths) const
 }
 
 DCPTime
-PlainText::full_length () const
+PlainTextContent::full_length () const
 {
        FrameRateChange const frc (active_video_frame_rate(), film()->video_frame_rate ());
        return DCPTime (_length, frc);
index 7a7ef66..02b4596 100644 (file)
@@ -25,14 +25,14 @@ class Job;
 /** @class PlainText
  *  @brief SubRip or SSA subtitles.
  */
-class PlainText : public Content
+class PlainTextContent : public Content
 {
 public:
-       PlainText (boost::shared_ptr<const Film>, boost::filesystem::path);
-       PlainText (boost::shared_ptr<const Film>, cxml::ConstNodePtr, int);
+       PlainTextContent (boost::shared_ptr<const Film>, boost::filesystem::path);
+       PlainTextContent (boost::shared_ptr<const Film>, cxml::ConstNodePtr, int);
 
-       boost::shared_ptr<PlainText> shared_from_this () {
-               return boost::dynamic_pointer_cast<PlainText> (Content::shared_from_this ());
+       boost::shared_ptr<PlainTextContent> shared_from_this () {
+               return boost::dynamic_pointer_cast<PlainTextContent> (Content::shared_from_this ());
        }
 
        void examine (boost::shared_ptr<Job>);
index 46a4bd5..005d306 100644 (file)
@@ -34,8 +34,8 @@ using boost::shared_ptr;
 using boost::optional;
 using boost::dynamic_pointer_cast;
 
-TextTextDecoder::TextTextDecoder (shared_ptr<const PlainText> content, shared_ptr<Log> log)
-       : TextSubtitle (content)
+PlainTextDecoder::PlainTextDecoder (shared_ptr<const PlainTextContent> content, shared_ptr<Log> log)
+       : PlainText (content)
        , _next (0)
 {
        ContentTime first;
@@ -46,7 +46,7 @@ TextTextDecoder::TextTextDecoder (shared_ptr<const PlainText> content, shared_pt
 }
 
 void
-TextTextDecoder::seek (ContentTime time, bool accurate)
+PlainTextDecoder::seek (ContentTime time, bool accurate)
 {
        /* It's worth back-tracking a little here as decoding is cheap and it's nice if we don't miss
           too many subtitles when seeking.
@@ -65,7 +65,7 @@ TextTextDecoder::seek (ContentTime time, bool accurate)
 }
 
 bool
-TextTextDecoder::pass ()
+PlainTextDecoder::pass ()
 {
        if (_next >= _subtitles.size ()) {
                return true;
@@ -79,7 +79,7 @@ TextTextDecoder::pass ()
 }
 
 ContentTimePeriod
-TextTextDecoder::content_time_period (sub::Subtitle s) const
+PlainTextDecoder::content_time_period (sub::Subtitle s) const
 {
        return ContentTimePeriod (
                ContentTime::from_seconds (s.from.all_as_seconds()),
index 046c5f8..f11fe3e 100644 (file)
 
 class PlainText;
 
-class TextTextDecoder : public Decoder, public TextSubtitle
+class PlainTextDecoder : public Decoder, public PlainText
 {
 public:
-       TextTextDecoder (boost::shared_ptr<const PlainText>, boost::shared_ptr<Log> log);
+       PlainTextDecoder (boost::shared_ptr<const PlainTextContent>, boost::shared_ptr<Log> log);
 
        void seek (ContentTime time, bool accurate);
        bool pass ();
index 578c9fa..8a24b21 100644 (file)
@@ -258,11 +258,11 @@ ContentPanel::selection_changed ()
        BOOST_FOREACH (shared_ptr<Content> i, selected ()) {
                DCPTime p;
                p = i->position();
-               if (dynamic_pointer_cast<PlainText>(i) && i->paths_valid()) {
+               if (dynamic_pointer_cast<PlainTextContent>(i) && i->paths_valid()) {
                        /* Rather special case; if we select a text subtitle file jump to its
                           first subtitle.
                        */
-                       TextSubtitle ts (dynamic_pointer_cast<PlainText>(i));
+                       PlainText ts (dynamic_pointer_cast<PlainTextContent>(i));
                        if (ts.first()) {
                                p += DCPTime(ts.first().get(), _film->active_frame_rate_change(i->position()));
                        }
index 65f98ed..e579621 100644 (file)
@@ -121,7 +121,7 @@ BOOST_AUTO_TEST_CASE (ffmpeg_encoder_prores_test6)
        film->set_container (Ratio::from_id ("185"));
        film->set_audio_channels (6);
 
-       shared_ptr<PlainText> s (new PlainText (film, "test/data/subrip2.srt"));
+       shared_ptr<PlainTextContent> s (new PlainTextContent (film, "test/data/subrip2.srt"));
        film->examine_and_add_content (s);
        BOOST_REQUIRE (!wait_for_jobs ());
        s->subtitle->set_colour (dcp::Colour (255, 255, 0));
@@ -146,7 +146,7 @@ BOOST_AUTO_TEST_CASE (ffmpeg_encoder_prores_test7)
        film->examine_and_add_content (c);
        BOOST_REQUIRE (!wait_for_jobs ());
 
-       shared_ptr<PlainText> s (new PlainText (film, "test/data/subrip.srt"));
+       shared_ptr<PlainTextContent> s (new PlainTextContent (film, "test/data/subrip.srt"));
        film->examine_and_add_content (s);
        BOOST_REQUIRE (!wait_for_jobs ());
        s->subtitle->set_colour (dcp::Colour (255, 255, 0));
@@ -172,7 +172,7 @@ BOOST_AUTO_TEST_CASE (ffmpeg_encoder_h264_test2)
        film->set_container (Ratio::from_id ("185"));
        film->set_audio_channels (6);
 
-       shared_ptr<PlainText> s (new PlainText (film, "test/data/subrip2.srt"));
+       shared_ptr<PlainTextContent> s (new PlainTextContent (film, "test/data/subrip2.srt"));
        film->examine_and_add_content (s);
        BOOST_REQUIRE (!wait_for_jobs ());
        s->subtitle->set_colour (dcp::Colour (255, 255, 0));
@@ -197,7 +197,7 @@ BOOST_AUTO_TEST_CASE (ffmpeg_encoder_h264_test3)
        film->examine_and_add_content (c);
        BOOST_REQUIRE (!wait_for_jobs ());
 
-       shared_ptr<PlainText> s (new PlainText (film, "test/data/subrip.srt"));
+       shared_ptr<PlainTextContent> s (new PlainTextContent (film, "test/data/subrip.srt"));
        film->examine_and_add_content (s);
        BOOST_REQUIRE (!wait_for_jobs ());
        s->subtitle->set_colour (dcp::Colour (255, 255, 0));
index 915dab4..7cc3bc2 100644 (file)
@@ -181,7 +181,7 @@ BOOST_AUTO_TEST_CASE (player_interleave_test)
        film->examine_and_add_content (c);
        BOOST_REQUIRE (!wait_for_jobs ());
 
-       shared_ptr<PlainText> s (new PlainText (film, "test/data/subrip.srt"));
+       shared_ptr<PlainTextContent> s (new PlainTextContent (film, "test/data/subrip.srt"));
        film->examine_and_add_content (s);
        BOOST_REQUIRE (!wait_for_jobs ());
 
index 2949adc..70552a7 100644 (file)
@@ -165,7 +165,7 @@ BOOST_AUTO_TEST_CASE (reels_test3)
 
        shared_ptr<Content> dcp (new DCPContent (film, "test/data/reels_test2"));
        film->examine_and_add_content (dcp);
-       shared_ptr<Content> sub (new PlainText (film, "test/data/subrip.srt"));
+       shared_ptr<Content> sub (new PlainTextContent (film, "test/data/subrip.srt"));
        film->examine_and_add_content (sub);
        wait_for_jobs ();
 
@@ -206,7 +206,7 @@ BOOST_AUTO_TEST_CASE (reels_test4)
                content[i]->video->set_length (24);
        }
 
-       shared_ptr<PlainText> subs (new PlainText (film, "test/data/subrip3.srt"));
+       shared_ptr<PlainTextContent> subs (new PlainTextContent (film, "test/data/subrip3.srt"));
        film->examine_and_add_content (subs);
        wait_for_jobs ();
 
index 1570cdc..e44e33c 100644 (file)
@@ -47,7 +47,7 @@ BOOST_AUTO_TEST_CASE (srt_subtitle_test)
        film->set_name ("frobozz");
        film->set_audio_channels (6);
        film->set_interop (false);
-       shared_ptr<PlainText> content (new PlainText (film, "test/data/subrip2.srt"));
+       shared_ptr<PlainTextContent> content (new PlainTextContent (film, "test/data/subrip2.srt"));
        film->examine_and_add_content (content);
        wait_for_jobs ();
 
@@ -69,7 +69,7 @@ BOOST_AUTO_TEST_CASE (srt_subtitle_test2)
        film->set_name ("frobozz");
        film->set_audio_channels (6);
        film->set_interop (false);
-       shared_ptr<PlainText> content (new PlainText (film, "test/data/subrip2.srt"));
+       shared_ptr<PlainTextContent> content (new PlainTextContent (film, "test/data/subrip2.srt"));
        film->examine_and_add_content (content);
        wait_for_jobs ();
 
@@ -104,7 +104,7 @@ BOOST_AUTO_TEST_CASE (srt_subtitle_test3)
        film->set_name ("frobozz");
        film->set_interop (true);
        film->set_audio_channels (6);
-       shared_ptr<PlainText> content (new PlainText (film, private_data / "Ankoemmling_short.srt"));
+       shared_ptr<PlainTextContent> content (new PlainTextContent (film, private_data / "Ankoemmling_short.srt"));
        film->examine_and_add_content (content);
        wait_for_jobs ();
 
@@ -125,7 +125,7 @@ BOOST_AUTO_TEST_CASE (srt_subtitle_test4)
        film->set_dcp_content_type (DCPContentType::from_isdcf_name ("TLR"));
        film->set_name ("frobozz");
        film->set_interop (false);
-       shared_ptr<PlainText> content (new PlainText (film, "test/data/subrip2.srt"));
+       shared_ptr<PlainTextContent> content (new PlainTextContent (film, "test/data/subrip2.srt"));
        content->subtitle->set_use (true);
        content->subtitle->set_burn (false);
        film->examine_and_add_content (content);
@@ -146,7 +146,7 @@ BOOST_AUTO_TEST_CASE (srt_subtitle_test5)
        film->set_name ("frobozz");
        film->set_interop (true);
        film->set_sequence (false);
-       shared_ptr<PlainText> content (new PlainText (film, "test/data/subrip2.srt"));
+       shared_ptr<PlainTextContent> content (new PlainTextContent (film, "test/data/subrip2.srt"));
        content->subtitle->set_use (true);
        content->subtitle->set_burn (false);
        film->examine_and_add_content (content);
@@ -164,7 +164,7 @@ BOOST_AUTO_TEST_CASE (srt_subtitle_test6)
 {
        shared_ptr<Film> film = new_test_film2 ("srt_subtitle_test6");
        film->set_interop (false);
-       shared_ptr<PlainText> content (new PlainText (film, "test/data/frames.srt"));
+       shared_ptr<PlainTextContent> content (new PlainTextContent (film, "test/data/frames.srt"));
        content->subtitle->set_use (true);
        content->subtitle->set_burn (false);
        film->examine_and_add_content (content);
index 7e2233c..ab15af1 100644 (file)
@@ -47,7 +47,7 @@ BOOST_AUTO_TEST_CASE (ssa_subtitle_test1)
        film->set_dcp_content_type (DCPContentType::from_isdcf_name ("TLR"));
        film->set_name ("frobozz");
        film->set_interop (true);
-       shared_ptr<PlainText> content (new PlainText (film, private_data / "DKH_UT_EN20160601def.ssa"));
+       shared_ptr<PlainTextContent> content (new PlainTextContent (film, private_data / "DKH_UT_EN20160601def.ssa"));
        film->examine_and_add_content (content);
        wait_for_jobs ();
 
index d150e90..317f93f 100644 (file)
@@ -43,7 +43,7 @@ BOOST_AUTO_TEST_CASE (subtitle_reel_number_test)
        film->set_container (Ratio::from_id ("185"));
        film->set_dcp_content_type (DCPContentType::from_isdcf_name ("TLR"));
        film->set_name ("frobozz");
-       shared_ptr<PlainText> content (new PlainText (film, "test/data/subrip5.srt"));
+       shared_ptr<PlainTextContent> content (new PlainTextContent (film, "test/data/subrip5.srt"));
        film->examine_and_add_content (content);
        BOOST_REQUIRE (!wait_for_jobs ());
        content->subtitle->set_use (true);