Prevent viewer updates on timeline drag (#175).
[dcpomatic.git] / src / lib / player.cc
index 18c42296f5f2b92964e15ce020b82813b979da60..50f401fb75fac7d3164ef3c80b10a654554575ea 100644 (file)
@@ -45,7 +45,7 @@ using boost::shared_ptr;
 using boost::weak_ptr;
 using boost::dynamic_pointer_cast;
 
-//#define DEBUG_PLAYER 1
+#define DEBUG_PLAYER 1
 
 class Piece
 {
@@ -97,7 +97,7 @@ Player::Player (shared_ptr<const Film> f, shared_ptr<const Playlist> p)
        , _audio_buffers (f->dcp_audio_channels(), 0)
 {
        _playlist->Changed.connect (bind (&Player::playlist_changed, this));
-       _playlist->ContentChanged.connect (bind (&Player::content_changed, this, _1, _2));
+       _playlist->ContentChanged.connect (bind (&Player::content_changed, this, _1, _2, _3));
        _film->Changed.connect (bind (&Player::film_changed, this, _1));
        set_video_container_size (_film->container()->size (_film->full_frame ()));
 }
@@ -138,7 +138,7 @@ Player::pass ()
                        continue;
                }
 
-               if (dynamic_pointer_cast<VideoDecoder> ((*i)->decoder)) {
+               if (_video && dynamic_pointer_cast<VideoDecoder> ((*i)->decoder)) {
                        if ((*i)->video_position < earliest_t) {
                                earliest_t = (*i)->video_position;
                                earliest = *i;
@@ -146,7 +146,7 @@ Player::pass ()
                        }
                }
 
-               if (dynamic_pointer_cast<AudioDecoder> ((*i)->decoder)) {
+               if (_audio && dynamic_pointer_cast<AudioDecoder> ((*i)->decoder)) {
                        if ((*i)->audio_position < earliest_t) {
                                earliest_t = (*i)->audio_position;
                                earliest = *i;
@@ -168,7 +168,7 @@ Player::pass ()
        case VIDEO:
                if (earliest_t > _video_position) {
 #ifdef DEBUG_PLAYER
-                       cout << "no video here; emitting black frame.\n";
+                       cout << "no video here; emitting black frame (earliest=" << earliest_t << ", video_position=" << _video_position << ").\n";
 #endif
                        emit_black ();
                } else {
@@ -232,12 +232,16 @@ Player::process_video (weak_ptr<Piece> weak_piece, shared_ptr<const Image> image
        if (image_size != _video_container_size) {
                assert (image_size.width <= _video_container_size.width);
                assert (image_size.height <= _video_container_size.height);
-               shared_ptr<Image> im (new SimpleImage (PIX_FMT_RGB24, _video_container_size, true));
+               shared_ptr<Image> im (new Image (PIX_FMT_RGB24, _video_container_size, true));
                im->make_black ();
                im->copy (work_image, Position<int> ((_video_container_size.width - image_size.width) / 2, (_video_container_size.height - image_size.height) / 2));
                work_image = im;
        }
 
+#ifdef DCPOMATIC_DEBUG
+       _last_video = piece->content;
+#endif 
+
         Video (work_image, same, time);
        time += TIME_HZ / _film->dcp_video_frame_rate();
 
@@ -260,12 +264,10 @@ Player::process_audio (weak_ptr<Piece> weak_piece, shared_ptr<const AudioBuffers
        shared_ptr<AudioContent> content = dynamic_pointer_cast<AudioContent> (piece->content);
        assert (content);
 
+       /* Resample */
        if (content->content_audio_frame_rate() != content->output_audio_frame_rate()) {
                shared_ptr<Resampler> r = resampler (content);
                audio = r->run (audio);
-               _last_resampler = r;
-       } else {
-               _last_resampler.reset ();
        }
 
        /* Remap channels */
@@ -273,20 +275,43 @@ Player::process_audio (weak_ptr<Piece> weak_piece, shared_ptr<const AudioBuffers
        dcp_mapped->make_silent ();
        list<pair<int, libdcp::Channel> > map = content->audio_mapping().content_to_dcp ();
        for (list<pair<int, libdcp::Channel> >::iterator i = map.begin(); i != map.end(); ++i) {
-               dcp_mapped->accumulate_channel (audio.get(), i->first, i->second);
+               if (i->first < audio->channels() && i->second < dcp_mapped->channels()) {
+                       dcp_mapped->accumulate_channel (audio.get(), i->first, i->second);
+               }
        }
 
        audio = dcp_mapped;
 
+       Time time = content->start() + (frame * TIME_HZ / _film->dcp_audio_frame_rate()) + (content->audio_delay() * TIME_HZ / 1000);
+
+       /* We must cut off anything that comes before the start of all time */
+       if (time < 0) {
+               int const frames = - time * _film->dcp_audio_frame_rate() / TIME_HZ;
+               if (frames >= audio->frames ()) {
+                       return;
+               }
+
+               shared_ptr<AudioBuffers> trimmed (new AudioBuffers (audio->channels(), audio->frames() - frames));
+               trimmed->copy_from (audio.get(), audio->frames() - frames, frames, 0);
+
+               audio = trimmed;
+               time = 0;
+       }
+
         /* The time of this audio may indicate that some of our buffered audio is not going to
            be added to any more, so it can be emitted.
         */
 
-       Time const time = content->start() + (frame * TIME_HZ / _film->dcp_audio_frame_rate());
-
         if (time > _audio_position) {
                 /* We can emit some audio from our buffers */
                 OutputAudioFrame const N = _film->time_to_audio_frames (time - _audio_position);
+               if (N > _audio_buffers.frames()) {
+                       /* We need some extra silence before whatever is in the buffers */
+                       _audio_buffers.ensure_size (N);
+                       _audio_buffers.move (0, N - _audio_buffers.frames(), _audio_buffers.frames ());
+                       _audio_buffers.make_silent (0, _audio_buffers.frames());
+                       _audio_buffers.set_frames (N);
+               }
                assert (N <= _audio_buffers.frames());
                 shared_ptr<AudioBuffers> emit (new AudioBuffers (_audio_buffers.channels(), N));
                 emit->copy_from (&_audio_buffers, N, 0, 0);
@@ -317,12 +342,6 @@ Player::flush ()
                _audio_buffers.set_frames (0);
        }
 
-       if (_last_resampler) {
-               shared_ptr<const AudioBuffers> resamp = _last_resampler->flush ();
-               Audio (resamp, _audio_position);
-               _audio_position += _film->audio_frames_to_time (resamp->frames ());
-       }
-       
        while (_video_position < _audio_position) {
                emit_black ();
        }
@@ -333,7 +352,10 @@ Player::flush ()
        
 }
 
-/** @return true on error */
+/** Seek so that the next pass() will yield (approximately) the requested frame.
+ *  Pass accurate = true to try harder to get close to the request.
+ *  @return true on error
+ */
 void
 Player::seek (Time t, bool accurate)
 {
@@ -356,11 +378,19 @@ Player::seek (Time t, bool accurate)
                s = max (static_cast<Time> (0), s);
                s = min (vc->length(), s);
 
+               (*i)->video_position = (*i)->audio_position = vc->start() + s;
+
                FrameRateConversion frc (vc->video_frame_rate(), _film->dcp_video_frame_rate());
-               VideoContent::Frame f = s * vc->video_frame_rate() / (frc.factor() * TIME_HZ);
+               /* Here we are converting from time (in the DCP) to a frame number in the content.
+                  Hence we need to use the DCP's frame rate and the double/skip correction, not
+                  the source's rate.
+               */
+               VideoContent::Frame f = s * _film->dcp_video_frame_rate() / (frc.factor() * TIME_HZ);
                dynamic_pointer_cast<VideoDecoder>((*i)->decoder)->seek (f, accurate);
        }
 
+       _video_position = _audio_position = t;
+       
        /* XXX: don't seek audio because we don't need to... */
 }
 
@@ -431,7 +461,7 @@ Player::setup_pieces ()
 }
 
 void
-Player::content_changed (weak_ptr<Content> w, int p)
+Player::content_changed (weak_ptr<Content> w, int property, bool frequent)
 {
        shared_ptr<Content> c = w.lock ();
        if (!c) {
@@ -439,16 +469,16 @@ Player::content_changed (weak_ptr<Content> w, int p)
        }
 
        if (
-               p == ContentProperty::START || p == ContentProperty::LENGTH ||
-               p == VideoContentProperty::VIDEO_CROP || p == VideoContentProperty::VIDEO_RATIO
+               property == ContentProperty::START || property == ContentProperty::LENGTH ||
+               property == VideoContentProperty::VIDEO_CROP || property == VideoContentProperty::VIDEO_RATIO
                ) {
                
                _have_valid_pieces = false;
-               Changed ();
+               Changed (frequent);
 
-       } else if (p == SubtitleContentProperty::SUBTITLE_OFFSET || p == SubtitleContentProperty::SUBTITLE_SCALE) {
+       } else if (property == SubtitleContentProperty::SUBTITLE_OFFSET || property == SubtitleContentProperty::SUBTITLE_SCALE) {
                update_subtitle ();
-               Changed ();
+               Changed (frequent);
        }
 }
 
@@ -456,14 +486,14 @@ void
 Player::playlist_changed ()
 {
        _have_valid_pieces = false;
-       Changed ();
+       Changed (false);
 }
 
 void
 Player::set_video_container_size (libdcp::Size s)
 {
        _video_container_size = s;
-       _black_frame.reset (new SimpleImage (PIX_FMT_RGB24, _video_container_size, true));
+       _black_frame.reset (new Image (PIX_FMT_RGB24, _video_container_size, true));
        _black_frame->make_black ();
 }
 
@@ -483,6 +513,10 @@ Player::resampler (shared_ptr<AudioContent> c)
 void
 Player::emit_black ()
 {
+#ifdef DCPOMATIC_DEBUG
+       _last_video.reset ();
+#endif
+       
        /* XXX: use same here */
        Video (_black_frame, false, _video_position);
        _video_position += _film->video_frames_to_time (1);
@@ -507,7 +541,7 @@ Player::film_changed (Film::Property p)
        */
 
        if (p == Film::SCALER || p == Film::WITH_SUBTITLES || p == Film::CONTAINER) {
-               Changed ();
+               Changed (false);
        }
 }