More enum class additions.
[dcpomatic.git] / src / lib / playlist.cc
index 48053bbf485aa893ddd903646ee911baaba05bdc..20534e467778b58246e87cb0ec5ab12033cfc09a 100644 (file)
@@ -34,8 +34,7 @@
 #include "compose.hpp"
 #include <libcxml/cxml.h>
 #include <libxml++/libxml++.h>
-#include <boost/shared_ptr.hpp>
-#include <boost/foreach.hpp>
+#include <boost/bind/placeholders.hpp>
 #include <iostream>
 
 #include "i18n.h"
@@ -48,10 +47,13 @@ using std::max;
 using std::string;
 using std::pair;
 using boost::optional;
-using boost::shared_ptr;
-using boost::weak_ptr;
-using boost::dynamic_pointer_cast;
+using std::shared_ptr;
+using std::weak_ptr;
+using std::dynamic_pointer_cast;
 using namespace dcpomatic;
+#if BOOST_VERSION >= 106100
+using namespace boost::placeholders;
+#endif
 
 Playlist::Playlist ()
        : _sequence (true)
@@ -70,17 +72,10 @@ Playlist::~Playlist ()
 void
 Playlist::content_change (weak_ptr<const Film> weak_film, ChangeType type, weak_ptr<Content> content, int property, bool frequent)
 {
-       /* Make sure we only hear about atomic changes (e.g. a PENDING always with the DONE/CANCELLED)
-          Ignore any DONE/CANCELLED that arrives without a PENDING.
-       */
-       if (_checker.send (type, property)) {
-               return;
-       }
-
-       shared_ptr<const Film> film = weak_film.lock ();
+       auto film = weak_film.lock ();
        DCPOMATIC_ASSERT (film);
 
-       if (type == CHANGE_TYPE_DONE) {
+       if (type == ChangeType::DONE) {
                if (
                        property == ContentProperty::TRIM_START ||
                        property == ContentProperty::TRIM_END ||
@@ -142,12 +137,12 @@ Playlist::maybe_sequence (shared_ptr<const Film> film)
 
        DCPTime next_left;
        DCPTime next_right;
-       BOOST_FOREACH (shared_ptr<Content> i, cont) {
+       for (auto i: cont) {
                if (!i->video) {
                        continue;
                }
 
-               if (i->video->frame_type() == VIDEO_FRAME_TYPE_3D_RIGHT) {
+               if (i->video->frame_type() == VideoFrameType::THREE_D_RIGHT) {
                        i->set_position (film, next_right);
                        next_right = i->end(film);
                } else {
@@ -161,7 +156,7 @@ Playlist::maybe_sequence (shared_ptr<const Film> film)
        /* Captions */
 
        DCPTime next;
-       BOOST_FOREACH (shared_ptr<Content> i, cont) {
+       for (auto i: cont) {
                if (i->text.empty() || find (placed.begin(), placed.end(), i) != placed.end()) {
                        continue;
                }
@@ -181,9 +176,9 @@ Playlist::video_identifier () const
 {
        string t;
 
-       BOOST_FOREACH (shared_ptr<const Content> i, content()) {
+       for (auto i: content()) {
                bool burn = false;
-               BOOST_FOREACH (shared_ptr<TextContent> j, i->text) {
+               for (auto j: i->text) {
                        if (j->burn()) {
                                burn = true;
                        }
@@ -208,11 +203,11 @@ Playlist::set_from_xml (shared_ptr<const Film> film, cxml::ConstNodePtr node, in
 {
        boost::mutex::scoped_lock lm (_mutex);
 
-       BOOST_FOREACH (cxml::NodePtr i, node->node_children ("Content")) {
-               shared_ptr<Content> content = content_factory (i, version, notes);
+       for (auto i: node->node_children ("Content")) {
+               auto content = content_factory (i, version, notes);
 
                /* See if this content should be nudged to start on a video frame */
-               DCPTime const old_pos = content->position();
+               auto const old_pos = content->position();
                content->set_position(film, old_pos);
                if (old_pos != content->position()) {
                        string note = _("Your project contains video content that was not aligned to a frame boundary.");
@@ -266,7 +261,7 @@ Playlist::set_from_xml (shared_ptr<const Film> film, cxml::ConstNodePtr node, in
 void
 Playlist::as_xml (xmlpp::Node* node, bool with_content_paths)
 {
-       BOOST_FOREACH (shared_ptr<Content> i, content()) {
+       for (auto i: content()) {
                i->as_xml (node->add_child ("Content"), with_content_paths);
        }
 }
@@ -274,7 +269,7 @@ Playlist::as_xml (xmlpp::Node* node, bool with_content_paths)
 void
 Playlist::add (shared_ptr<const Film> film, shared_ptr<Content> c)
 {
-       Change (CHANGE_TYPE_PENDING);
+       Change (ChangeType::PENDING);
 
        {
                boost::mutex::scoped_lock lm (_mutex);
@@ -283,7 +278,7 @@ Playlist::add (shared_ptr<const Film> film, shared_ptr<Content> c)
                reconnect (film);
        }
 
-       Change (CHANGE_TYPE_DONE);
+       Change (ChangeType::DONE);
 
        LengthChange ();
 }
@@ -291,14 +286,14 @@ Playlist::add (shared_ptr<const Film> film, shared_ptr<Content> c)
 void
 Playlist::remove (shared_ptr<Content> c)
 {
-       Change (CHANGE_TYPE_PENDING);
+       Change (ChangeType::PENDING);
 
        bool cancelled = false;
 
        {
                boost::mutex::scoped_lock lm (_mutex);
 
-               ContentList::iterator i = _content.begin ();
+               auto i = _content.begin ();
                while (i != _content.end() && *i != c) {
                        ++i;
                }
@@ -311,9 +306,9 @@ Playlist::remove (shared_ptr<Content> c)
        }
 
        if (cancelled) {
-               Change (CHANGE_TYPE_CANCELLED);
+               Change (ChangeType::CANCELLED);
        } else {
-               Change (CHANGE_TYPE_DONE);
+               Change (ChangeType::DONE);
        }
 
        /* This won't change order, so it does not need a sort */
@@ -324,12 +319,12 @@ Playlist::remove (shared_ptr<Content> c)
 void
 Playlist::remove (ContentList c)
 {
-       Change (CHANGE_TYPE_PENDING);
+       Change (ChangeType::PENDING);
 
        {
                boost::mutex::scoped_lock lm (_mutex);
 
-               BOOST_FOREACH (shared_ptr<Content> i, c) {
+               for (auto i: c) {
                        ContentList::iterator j = _content.begin ();
                        while (j != _content.end() && *j != i) {
                                ++j;
@@ -341,7 +336,7 @@ Playlist::remove (ContentList c)
                }
        }
 
-       Change (CHANGE_TYPE_DONE);
+       Change (ChangeType::DONE);
 
        /* This won't change order, so it does not need a sort */
 
@@ -364,18 +359,18 @@ public:
 int
 Playlist::best_video_frame_rate () const
 {
-       list<int> const allowed_dcp_frame_rates = Config::instance()->allowed_dcp_frame_rates ();
+       auto const allowed_dcp_frame_rates = Config::instance()->allowed_dcp_frame_rates ();
 
        /* Work out what rates we could manage, including those achieved by using skip / repeat */
        list<FrameRateCandidate> candidates;
 
        /* Start with the ones without skip / repeat so they will get matched in preference to skipped/repeated ones */
-       BOOST_FOREACH (int i, allowed_dcp_frame_rates) {
+       for (auto i: allowed_dcp_frame_rates) {
                candidates.push_back (FrameRateCandidate(i, i));
        }
 
        /* Then the skip/repeat ones */
-       BOOST_FOREACH (int i, allowed_dcp_frame_rates) {
+       for (auto i: allowed_dcp_frame_rates) {
                candidates.push_back (FrameRateCandidate (float(i) / 2, i));
                candidates.push_back (FrameRateCandidate (float(i) * 2, i));
        }
@@ -383,11 +378,11 @@ Playlist::best_video_frame_rate () const
        /* Pick the best one */
        float error = std::numeric_limits<float>::max ();
        optional<FrameRateCandidate> best;
-       list<FrameRateCandidate>::iterator i = candidates.begin();
+       auto i = candidates.begin();
        while (i != candidates.end()) {
 
                float this_error = 0;
-               BOOST_FOREACH (shared_ptr<Content> j, content()) {
+               for (auto j: content()) {
                        if (!j->video || !j->video_frame_rate()) {
                                continue;
                        }
@@ -422,7 +417,7 @@ DCPTime
 Playlist::length (shared_ptr<const Film> film) const
 {
        DCPTime len;
-       BOOST_FOREACH (shared_ptr<const Content> i, content()) {
+       for (auto i: content()) {
                len = max (len, i->end(film));
        }
 
@@ -433,13 +428,13 @@ Playlist::length (shared_ptr<const Film> film) const
 optional<DCPTime>
 Playlist::start () const
 {
-       ContentList cont = content ();
+       auto cont = content ();
        if (cont.empty()) {
-               return optional<DCPTime> ();
+               return {};
        }
 
-       DCPTime start = DCPTime::max ();
-       BOOST_FOREACH (shared_ptr<Content> i, cont) {
+       auto start = DCPTime::max ();
+       for (auto i: cont) {
                start = min (start, i->position ());
        }
 
@@ -450,7 +445,7 @@ Playlist::start () const
 void
 Playlist::disconnect ()
 {
-       BOOST_FOREACH (boost::signals2::connection& i, _content_connections) {
+       for (auto& i: _content_connections) {
                i.disconnect ();
        }
 
@@ -463,7 +458,7 @@ Playlist::reconnect (shared_ptr<const Film> film)
 {
        disconnect ();
 
-       BOOST_FOREACH (shared_ptr<Content> i, _content) {
+       for (auto i: _content) {
                _content_connections.push_back (i->Change.connect(boost::bind(&Playlist::content_change, this, film, _1, _2, _3, _4)));
        }
 }
@@ -472,7 +467,7 @@ DCPTime
 Playlist::video_end (shared_ptr<const Film> film) const
 {
        DCPTime end;
-       BOOST_FOREACH (shared_ptr<Content> i, content()) {
+       for (auto i: content()) {
                if (i->video) {
                        end = max (end, i->end(film));
                }
@@ -485,7 +480,7 @@ DCPTime
 Playlist::text_end (shared_ptr<const Film> film) const
 {
        DCPTime end;
-       BOOST_FOREACH (shared_ptr<Content> i, content()) {
+       for (auto i: content()) {
                if (!i->text.empty ()) {
                        end = max (end, i->end(film));
                }
@@ -556,22 +551,22 @@ void
 Playlist::repeat (shared_ptr<const Film> film, ContentList c, int n)
 {
        pair<DCPTime, DCPTime> range (DCPTime::max (), DCPTime ());
-       BOOST_FOREACH (shared_ptr<Content> i, c) {
+       for (auto i: c) {
                range.first = min (range.first, i->position ());
                range.second = max (range.second, i->position ());
                range.first = min (range.first, i->end(film));
                range.second = max (range.second, i->end(film));
        }
 
-       Change (CHANGE_TYPE_PENDING);
+       Change (ChangeType::PENDING);
 
        {
                boost::mutex::scoped_lock lm (_mutex);
 
                DCPTime pos = range.second;
                for (int i = 0; i < n; ++i) {
-                       BOOST_FOREACH (shared_ptr<Content> j, c) {
-                               shared_ptr<Content> copy = j->clone ();
+                       for (auto j: c) {
+                               auto copy = j->clone ();
                                copy->set_position (film, pos + copy->position() - range.first);
                                _content.push_back (copy);
                        }
@@ -582,15 +577,15 @@ Playlist::repeat (shared_ptr<const Film> film, ContentList c, int n)
                reconnect (film);
        }
 
-       Change (CHANGE_TYPE_DONE);
+       Change (ChangeType::DONE);
 }
 
 void
 Playlist::move_earlier (shared_ptr<const Film> film, shared_ptr<Content> c)
 {
-       ContentList cont = content ();
-       ContentList::iterator previous = cont.end();
-       ContentList::iterator i = cont.begin();
+       auto cont = content ();
+       auto previous = cont.end();
+       auto i = cont.begin();
        while (i != cont.end() && *i != c) {
                previous = i;
                ++i;
@@ -601,9 +596,9 @@ Playlist::move_earlier (shared_ptr<const Film> film, shared_ptr<Content> c)
                return;
        }
 
-       shared_ptr<Content> previous_c = *previous;
+       auto previous_c = *previous;
 
-       DCPTime const p = previous_c->position ();
+       auto const p = previous_c->position ();
        previous_c->set_position (film, p + c->length_after_trim(film));
        c->set_position (film, p);
 }
@@ -611,8 +606,8 @@ Playlist::move_earlier (shared_ptr<const Film> film, shared_ptr<Content> c)
 void
 Playlist::move_later (shared_ptr<const Film> film, shared_ptr<Content> c)
 {
-       ContentList cont = content ();
-       ContentList::iterator i = cont.begin();
+       auto cont = content ();
+       auto i = cont.begin();
        while (i != cont.end() && *i != c) {
                ++i;
        }
@@ -626,7 +621,7 @@ Playlist::move_later (shared_ptr<const Film> film, shared_ptr<Content> c)
                return;
        }
 
-       shared_ptr<Content> next_c = *next;
+       auto next_c = *next;
 
        next_c->set_position (film, c->position());
        c->set_position (film, c->position() + next_c->length_after_trim(film));
@@ -638,8 +633,8 @@ Playlist::required_disk_space (shared_ptr<const Film> film, int j2k_bandwidth, i
        int64_t video = uint64_t (j2k_bandwidth / 8) * length(film).seconds();
        int64_t audio = uint64_t (audio_channels * audio_frame_rate * 3) * length(film).seconds();
 
-       BOOST_FOREACH (shared_ptr<Content> i, content()) {
-               shared_ptr<DCPContent> d = dynamic_pointer_cast<DCPContent> (i);
+       for (auto i: content()) {
+               auto d = dynamic_pointer_cast<DCPContent> (i);
                if (d) {
                        if (d->reference_video()) {
                                video -= uint64_t (j2k_bandwidth / 8) * d->length_after_trim(film).seconds();
@@ -659,9 +654,9 @@ Playlist::content_summary (shared_ptr<const Film> film, DCPTimePeriod period) co
 {
        string best_summary;
        int best_score = -1;
-       BOOST_FOREACH (shared_ptr<Content> i, content()) {
+       for (auto i: content()) {
                int score = 0;
-               optional<DCPTimePeriod> const o = DCPTimePeriod(i->position(), i->end(film)).overlap (period);
+               auto const o = DCPTimePeriod(i->position(), i->end(film)).overlap (period);
                if (o) {
                        score += 100 * o.get().duration().get() / period.duration().get();
                }
@@ -684,7 +679,7 @@ Playlist::speed_up_range (int dcp_video_frame_rate) const
 {
        pair<double, double> range (DBL_MAX, -DBL_MAX);
 
-       BOOST_FOREACH (shared_ptr<Content> i, content()) {
+       for (auto i: content()) {
                if (!i->video) {
                        continue;
                }