using std::cout;
using boost::optional;
-using boost::shared_ptr;
-using boost::weak_ptr;
+using std::shared_ptr;
+using std::weak_ptr;
+using namespace dcpomatic;
Decoder::Decoder (weak_ptr<const Film> film)
- : _film (film)
+ : WeakConstFilm (film)
{
}
optional<ContentTime> pos;
shared_ptr<const Film> f = film();
- if (video && !video->ignore() && (!pos || video->position(f) < *pos)) {
+ if (video && !video->ignore() && (!pos || video->position(f).get_value_or(ContentTime()) < *pos)) {
pos = video->position(f);
}
- if (audio && !audio->ignore() && (!pos || audio->position(f) < *pos)) {
+ if (audio && !audio->ignore() && (!pos || audio->position(f).get_value_or(ContentTime()) < *pos)) {
pos = audio->position(f);
}
- BOOST_FOREACH (shared_ptr<TextDecoder> i, text) {
- if (!i->ignore() && (!pos || i->position(f) < *pos)) {
- pos = i->position(f);
+ /* Only decide position based on subtitle sources if there is nothing else
+ to go on. Otherwise we can have problems with muxed sources which have
+ (for example) video, audio and a subtitle. If the subtitle data runs out
+ before the video/audio the position() call will return the position of the
+ end of the subs. This causes this file to be pass()ed in favour of others,
+ which can cause bugs like #1581.
+ */
+ if (!pos) {
+ for (auto i: text) {
+ if (!i->ignore() && (!pos || i->position(f) < *pos)) {
+ pos = i->position(f);
+ }
}
}
if (audio) {
audio->seek ();
}
- BOOST_FOREACH (shared_ptr<TextDecoder> i, text) {
+ for (auto i: text) {
i->seek ();
}
}
}
return text.front ();
}
-
-shared_ptr<const Film>
-Decoder::film () const
-{
- shared_ptr<const Film> f = _film.lock ();
- DCPOMATIC_ASSERT (f);
- return f;
-}