Merge branch 'master' of ssh://git.carlh.net/home/carl/git/dcpomatic

[dcpomatic.git] / src / lib / dcp_decoder.cc

diff --git a/src/lib/dcp_decoder.cc b/src/lib/dcp_decoder.cc
index 14b791aa2a48a9ef60579c09d8ed02e4461c59ef..5f06501b621f2c369e1c8d8ed3b5e1d6a69df01d 100644 (file)
--- a/src/lib/dcp_decoder.cc
+++ b/src/lib/dcp_decoder.cc
@@ -54,7 +54,9 @@ DCPDecoder::DCPDecoder (shared_ptr<const DCPContent> c, shared_ptr<Log> log, boo
        : DCP (c)
        , _decode_referenced (false)
 {
-       video.reset (new VideoDecoder (this, c, log));
+       if (c->video) {
+               video.reset (new VideoDecoder (this, c, log));
+       }
        if (c->audio) {
                audio.reset (new AudioDecoder (this, c->audio, log, fast));
        }
@@ -99,6 +101,11 @@ DCPDecoder::pass ()
        /* Frame within the (played part of the) reel that is coming up next */
        int64_t const frame = _next.frames_round (vfr);
 
+       /* We must emit subtitles first as when we emit the video for this frame
+          it will expect already to have the subs.
+       */
+       pass_subtitles (_next);
+
        if ((_mono_reader || _stereo_reader) && (_decode_referenced || !_dcp_content->reference_video())) {
                shared_ptr<dcp::PictureAsset> asset = (*_reel)->main_picture()->asset ();
                int64_t const entry_point = (*_reel)->main_picture()->entry_point ();
@@ -162,8 +169,6 @@ DCPDecoder::pass ()
                audio->emit (_dcp_content->audio->stream(), data, ContentTime::from_frames (_offset, vfr) + _next);
        }
 
-       pass_subtitles (_next);
-
        _next += ContentTime::from_frames (1, vfr);
 
        if ((*_reel)->main_picture ()) {
@@ -250,36 +255,40 @@ DCPDecoder::get_readers ()
 void
 DCPDecoder::seek (ContentTime t, bool accurate)
 {
+       if (!_dcp_content->can_be_played ()) {
+               return;
+       }
+
        Decoder::seek (t, accurate);
 
        _reel = _reels.begin ();
        _offset = 0;
        get_readers ();
 
-       if (accurate) {
-               int const pre_roll_seconds = 2;
+       int const pre_roll_seconds = 2;
 
-               /* Pre-roll for subs */
+       /* Pre-roll for subs */
 
-               ContentTime pre = t - ContentTime::from_seconds (pre_roll_seconds);
-               if (pre < ContentTime()) {
-                       pre = ContentTime ();
-               }
+       ContentTime pre = t - ContentTime::from_seconds (pre_roll_seconds);
+       if (pre < ContentTime()) {
+               pre = ContentTime ();
+       }
 
-               /* Seek to pre-roll position */
+       /* Seek to pre-roll position */
 
-               while (_reel != _reels.end() && pre >= ContentTime::from_frames ((*_reel)->main_picture()->duration(), _dcp_content->active_video_frame_rate ())) {
-                       pre -= ContentTime::from_frames ((*_reel)->main_picture()->duration(), _dcp_content->active_video_frame_rate ());
-                       next_reel ();
-               }
+       while (_reel != _reels.end() && pre >= ContentTime::from_frames ((*_reel)->main_picture()->duration(), _dcp_content->active_video_frame_rate ())) {
+               ContentTime rd = ContentTime::from_frames ((*_reel)->main_picture()->duration(), _dcp_content->active_video_frame_rate ());
+               pre -= rd;
+               t -= rd;
+               next_reel ();
+       }
 
-               /* Pass subtitles in the pre-roll */
+       /* Pass subtitles in the pre-roll */
 
-               double const vfr = _dcp_content->active_video_frame_rate ();
-               for (int i = 0; i < pre_roll_seconds * vfr; ++i) {
-                       pass_subtitles (pre);
-                       pre += ContentTime::from_frames (1, vfr);
-               }
+       double const vfr = _dcp_content->active_video_frame_rate ();
+       for (int i = 0; i < pre_roll_seconds * vfr; ++i) {
+               pass_subtitles (pre);
+               pre += ContentTime::from_frames (1, vfr);
        }
 
        /* Seek to correct position */
@@ -297,8 +306,12 @@ DCPDecoder::set_decode_referenced (bool r)
 {
        _decode_referenced = r;
 
-       video->set_ignore (_dcp_content->reference_video() && !_decode_referenced);
-       audio->set_ignore (_dcp_content->reference_audio() && !_decode_referenced);
+       if (video) {
+               video->set_ignore (_dcp_content->reference_video() && !_decode_referenced);
+       }
+       if (audio) {
+               audio->set_ignore (_dcp_content->reference_audio() && !_decode_referenced);
+       }
 }
 
 void