X-Git-Url: https://main.carlh.net/gitweb/?p=dcpomatic.git;a=blobdiff_plain;f=src%2Fwx%2Ffilm_viewer.cc;h=0e47ac1acb47947da199f71430e03d2657c70e07;hp=9c714b5622ccc79cbd2bfdaa977f17626d61f0be;hb=a5be11a965c2c38442e4e069874e7e21b5b43a5c;hpb=bd763364d9c96928f3bd6bd5fe46ebe1a06e6d0a diff --git a/src/wx/film_viewer.cc b/src/wx/film_viewer.cc index 9c714b562..0e47ac1ac 100644 --- a/src/wx/film_viewer.cc +++ b/src/wx/film_viewer.cc @@ -1,5 +1,5 @@ /* - Copyright (C) 2012-2017 Carl Hetherington + Copyright (C) 2012-2019 Carl Hetherington This file is part of DCP-o-matic. @@ -26,7 +26,7 @@ #include "playhead_to_timecode_dialog.h" #include "playhead_to_frame_dialog.h" #include "wx_util.h" -#include "closed_captions_view.h" +#include "closed_captions_dialog.h" #include "lib/film.h" #include "lib/ratio.h" #include "lib/util.h" @@ -43,6 +43,8 @@ #include "lib/butler.h" #include "lib/log.h" #include "lib/config.h" +#include "lib/compose.hpp" +#include "lib/dcpomatic_log.h" extern "C" { #include } @@ -73,110 +75,37 @@ rtaudio_callback (void* out, void *, unsigned int frames, double, RtAudioStreamS return reinterpret_cast(data)->audio_callback (out, frames); } -FilmViewer::FilmViewer (wxWindow* p, bool outline_content, bool jump_to_selected) - : wxPanel (p) - , _panel (new wxPanel (this)) - , _outline_content (0) - , _eye (0) - , _jump_to_selected (0) - , _slider (new wxSlider (this, wxID_ANY, 0, 0, 4096)) - , _rewind_button (new wxButton (this, wxID_ANY, wxT("|<"))) - , _back_button (new wxButton (this, wxID_ANY, wxT("<"))) - , _forward_button (new wxButton (this, wxID_ANY, wxT(">"))) - , _frame_number (new wxStaticText (this, wxID_ANY, wxT(""))) - , _timecode (new wxStaticText (this, wxID_ANY, wxT(""))) - , _play_button (new wxToggleButton (this, wxID_ANY, _("Play"))) +FilmViewer::FilmViewer (wxWindow* p) + : _panel (new wxPanel (p)) , _coalesce_player_changes (false) - , _slider_being_moved (false) - , _was_running_before_slider (false) , _audio (DCPOMATIC_RTAUDIO_API) , _audio_channels (0) , _audio_block_size (1024) , _playing (false) , _latency_history_count (0) , _dropped (0) - , _closed_captions_dialog (new ClosedCaptionsDialog(GetParent())) + , _closed_captions_dialog (new ClosedCaptionsDialog(p)) + , _outline_content (false) + , _eyes (EYES_LEFT) + , _pad_black (false) +#ifdef DCPOMATIC_VARIANT_SWAROOP + , _in_watermark (false) + , _background_image (false) +#endif { #ifndef __WXOSX__ _panel->SetDoubleBuffered (true); #endif _panel->SetBackgroundStyle (wxBG_STYLE_PAINT); + _panel->SetBackgroundColour (*wxBLACK); - _v_sizer = new wxBoxSizer (wxVERTICAL); - SetSizer (_v_sizer); - - _v_sizer->Add (_panel, 1, wxEXPAND); - - wxBoxSizer* view_options = new wxBoxSizer (wxHORIZONTAL); - if (outline_content) { - _outline_content = new wxCheckBox (this, wxID_ANY, _("Outline content")); - view_options->Add (_outline_content, 0, wxRIGHT | wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_GAP); - } - - _eye = new wxChoice (this, wxID_ANY); - _eye->Append (_("Left")); - _eye->Append (_("Right")); - _eye->SetSelection (0); - view_options->Add (_eye, 0, wxLEFT | wxRIGHT | wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_GAP); - - if (jump_to_selected) { - _jump_to_selected = new wxCheckBox (this, wxID_ANY, _("Jump to selected content")); - view_options->Add (_jump_to_selected, 0, wxLEFT | wxRIGHT | wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_GAP); - } - - _v_sizer->Add (view_options, 0, wxALL, DCPOMATIC_SIZER_GAP); - - wxBoxSizer* h_sizer = new wxBoxSizer (wxHORIZONTAL); - - wxBoxSizer* time_sizer = new wxBoxSizer (wxVERTICAL); - time_sizer->Add (_frame_number, 0, wxEXPAND); - time_sizer->Add (_timecode, 0, wxEXPAND); - - h_sizer->Add (_rewind_button, 0, wxALL, 2); - h_sizer->Add (_back_button, 0, wxALL, 2); - h_sizer->Add (time_sizer, 0, wxEXPAND); - h_sizer->Add (_forward_button, 0, wxALL, 2); - h_sizer->Add (_play_button, 0, wxEXPAND); - h_sizer->Add (_slider, 1, wxEXPAND); - - _v_sizer->Add (h_sizer, 0, wxEXPAND | wxALL, 6); - - _frame_number->SetMinSize (wxSize (84, -1)); - _rewind_button->SetMinSize (wxSize (32, -1)); - _back_button->SetMinSize (wxSize (32, -1)); - _forward_button->SetMinSize (wxSize (32, -1)); - - _panel->Bind (wxEVT_PAINT, boost::bind (&FilmViewer::paint_panel, this)); - _panel->Bind (wxEVT_SIZE, boost::bind (&FilmViewer::panel_sized, this, _1)); - if (_outline_content) { - _outline_content->Bind (wxEVT_CHECKBOX, boost::bind (&FilmViewer::refresh_panel, this)); - } - _eye->Bind (wxEVT_CHOICE, boost::bind (&FilmViewer::slow_refresh, this)); - _slider->Bind (wxEVT_SCROLL_THUMBTRACK, boost::bind (&FilmViewer::slider_moved, this, false)); - _slider->Bind (wxEVT_SCROLL_PAGEUP, boost::bind (&FilmViewer::slider_moved, this, true)); - _slider->Bind (wxEVT_SCROLL_PAGEDOWN, boost::bind (&FilmViewer::slider_moved, this, true)); - _slider->Bind (wxEVT_SCROLL_THUMBRELEASE, boost::bind (&FilmViewer::slider_released, this)); - _play_button->Bind (wxEVT_TOGGLEBUTTON, boost::bind (&FilmViewer::play_clicked, this)); - _timer.Bind (wxEVT_TIMER, boost::bind (&FilmViewer::timer, this)); - _rewind_button->Bind (wxEVT_LEFT_DOWN, boost::bind (&FilmViewer::rewind_clicked, this, _1)); - _back_button->Bind (wxEVT_LEFT_DOWN, boost::bind (&FilmViewer::back_clicked, this, _1)); - _forward_button->Bind (wxEVT_LEFT_DOWN, boost::bind (&FilmViewer::forward_clicked, this, _1)); - _frame_number->Bind (wxEVT_LEFT_DOWN, boost::bind (&FilmViewer::frame_number_clicked, this)); - _timecode->Bind (wxEVT_LEFT_DOWN, boost::bind (&FilmViewer::timecode_clicked, this)); - if (_jump_to_selected) { - _jump_to_selected->Bind (wxEVT_CHECKBOX, boost::bind (&FilmViewer::jump_to_selected_clicked, this)); - _jump_to_selected->SetValue (Config::instance()->jump_to_selected ()); - } + _panel->Bind (wxEVT_PAINT, boost::bind (&FilmViewer::paint_panel, this)); + _panel->Bind (wxEVT_SIZE, boost::bind (&FilmViewer::panel_sized, this, _1)); + _timer.Bind (wxEVT_TIMER, boost::bind (&FilmViewer::timer, this)); set_film (shared_ptr ()); - JobManager::instance()->ActiveJobsChanged.connect ( - bind (&FilmViewer::active_jobs_changed, this, _2) - ); - - setup_sensitivity (); - _config_changed_connection = Config::instance()->Changed.connect (bind (&FilmViewer::config_changed, this, _1)); config_changed (Config::SOUND_OUTPUT); } @@ -194,13 +123,13 @@ FilmViewer::set_film (shared_ptr film) } _film = film; + _video_position = DCPTime (); + _player_video.first.reset (); + _player_video.second = DCPTime (); _frame.reset (); _closed_captions_dialog->clear (); - update_position_slider (); - update_position_label (); - if (!_film) { _player.reset (); recreate_butler (); @@ -215,18 +144,17 @@ FilmViewer::set_film (shared_ptr film) if (_dcp_decode_reduction) { _player->set_dcp_decode_reduction (_dcp_decode_reduction); } - } catch (bad_alloc) { - error_dialog (this, _("There is not enough free memory to do that.")); + } catch (bad_alloc &) { + error_dialog (_panel, _("There is not enough free memory to do that.")); _film.reset (); return; } - _player->set_always_burn_open_captions (); + _player->set_always_burn_open_subtitles (); _player->set_play_referenced (); - _film->Changed.connect (boost::bind (&FilmViewer::film_changed, this, _1)); - _player->Changed.connect (boost::bind (&FilmViewer::player_changed, this, _1, _2)); - _player->Caption.connect (boost::bind (&FilmViewer::caption, this, _1, _2, _3)); + _film->Change.connect (boost::bind (&FilmViewer::film_change, this, _1, _2)); + _player->Change.connect (boost::bind (&FilmViewer::player_change, this, _1, _2, _3)); /* Keep about 1 second's worth of history samples */ _latency_history_count = _film->audio_frame_rate() / _audio_block_size; @@ -235,8 +163,6 @@ FilmViewer::set_film (shared_ptr film) calculate_sizes (); slow_refresh (); - - setup_sensitivity (); } void @@ -270,11 +196,13 @@ FilmViewer::recreate_butler () map.set (dcp::RS, 1, 1 / sqrt(2)); // Rs -> Rt } - _butler.reset (new Butler (_player, _film->log(), map, _audio_channels)); + _butler.reset (new Butler(_player, map, _audio_channels, bind(&PlayerVideo::force, _1, AV_PIX_FMT_RGB24), false, true)); if (!Config::instance()->sound() && !_audio.isStreamOpen()) { _butler->disable_audio (); } + _closed_captions_dialog->set_butler (_butler); + if (was_running) { start (); } @@ -293,10 +221,17 @@ FilmViewer::get () DCPOMATIC_ASSERT (_butler); do { - _player_video = _butler->get_video (); + Butler::Error e; + _player_video = _butler->get_video (&e); + if (!_player_video.first && e == Butler::AGAIN) { + signal_manager->when_idle (boost::bind(&FilmViewer::get, this)); + return; + } } while ( + _player_video.first && _film->three_d() && - ((_eye->GetSelection() == 0 && _player_video.first->eyes() == EYES_RIGHT) || (_eye->GetSelection() == 1 && _player_video.first->eyes() == EYES_LEFT)) + _eyes != _player_video.first->eyes() && + _player_video.first->eyes() != EYES_BOTH ); _butler->rethrow (); @@ -336,15 +271,11 @@ FilmViewer::display_player_video () * The content's specified colour conversion indicates the colourspace * which the content is in (according to the user). * - * PlayerVideo::image (bound to PlayerVideo::always_rgb) will take the source + * PlayerVideo::image (bound to PlayerVideo::force) will take the source * image and convert it (from whatever the user has said it is) to RGB. */ - _frame = _player_video.first->image ( - bind (&Log::dcp_log, _film->log().get(), _1, _2), - bind (&PlayerVideo::always_rgb, _1), - false, true - ); + _frame = _player_video.first->image (bind(&PlayerVideo::force, _1, AV_PIX_FMT_RGB24), false, true); ImageChanged (_player_video.first); @@ -354,7 +285,7 @@ FilmViewer::display_player_video () refresh_panel (); - _closed_captions_dialog->refresh (time()); + _closed_captions_dialog->update (time()); } void @@ -365,14 +296,16 @@ FilmViewer::timer () } get (); - update_position_label (); - update_position_slider (); + PositionChanged (); DCPTime const next = _video_position + one_video_frame(); if (next >= _film->length()) { stop (); + Finished (); + return; } + LOG_DEBUG_PLAYER("%1 -> %2; delay %3", next.seconds(), time().seconds(), max((next.seconds() - time().seconds()) * 1000, 1.0)); _timer.Start (max ((next.seconds() - time().seconds()) * 1000, 1.0), wxTIMER_ONE_SHOT); if (_butler) { @@ -380,76 +313,109 @@ FilmViewer::timer () } } +bool +#ifdef DCPOMATIC_VARIANT_SWAROOP +FilmViewer::maybe_draw_background_image (wxPaintDC& dc) +{ + optional bg = Config::instance()->player_background_image(); + if (bg) { + wxImage image (std_to_wx(bg->string())); + wxBitmap bitmap (image); + dc.DrawBitmap (bitmap, max(0, (_panel_size.width - image.GetSize().GetWidth()) / 2), max(0, (_panel_size.height - image.GetSize().GetHeight()) / 2)); + return true; + } + + return false; +} +#else +FilmViewer::maybe_draw_background_image (wxPaintDC &) +{ + return false; +} +#endif + void FilmViewer::paint_panel () { wxPaintDC dc (_panel); - if (!_frame || !_film || !_out_size.width || !_out_size.height || _out_size != _frame->size()) { +#ifdef DCPOMATIC_VARIANT_SWAROOP + if (_background_image) { + dc.Clear (); + maybe_draw_background_image (dc); + return; + } +#endif + + if (!_out_size.width || !_out_size.height || !_film || !_frame || _out_size != _frame->size()) { dc.Clear (); return; } wxImage frame (_out_size.width, _out_size.height, _frame->data()[0], true); wxBitmap frame_bitmap (frame); - dc.DrawBitmap (frame_bitmap, 0, 0); + dc.DrawBitmap (frame_bitmap, 0, max(0, (_panel_size.height - _out_size.height) / 2)); + +#ifdef DCPOMATIC_VARIANT_SWAROOP + DCPTime const period = DCPTime::from_seconds(Config::instance()->player_watermark_period() * 60); + int64_t n = _video_position.get() / period.get(); + DCPTime from(n * period.get()); + DCPTime to = from + DCPTime::from_seconds(Config::instance()->player_watermark_duration() / 1000.0); + if (from <= _video_position && _video_position <= to) { + if (!_in_watermark) { + _in_watermark = true; + _watermark_x = rand() % _panel_size.width; + _watermark_y = rand() % _panel_size.height; + } + dc.SetTextForeground(*wxWHITE); + string wm = Config::instance()->player_watermark_theatre(); + boost::posix_time::ptime t = boost::posix_time::second_clock::local_time(); + wm += "\n" + boost::posix_time::to_iso_extended_string(t); + dc.DrawText(std_to_wx(wm), _watermark_x, _watermark_y); + } else { + _in_watermark = false; + } +#endif if (_out_size.width < _panel_size.width) { - wxPen p (GetBackgroundColour ()); - wxBrush b (GetBackgroundColour ()); + /* XXX: these colours are right for GNOME; may need adjusting for other OS */ + wxPen p (_pad_black ? wxColour(0, 0, 0) : wxColour(240, 240, 240)); + wxBrush b (_pad_black ? wxColour(0, 0, 0) : wxColour(240, 240, 240)); dc.SetPen (p); dc.SetBrush (b); dc.DrawRectangle (_out_size.width, 0, _panel_size.width - _out_size.width, _panel_size.height); } if (_out_size.height < _panel_size.height) { - wxPen p (GetBackgroundColour ()); - wxBrush b (GetBackgroundColour ()); + wxPen p (_pad_black ? wxColour(0, 0, 0) : wxColour(240, 240, 240)); + wxBrush b (_pad_black ? wxColour(0, 0, 0) : wxColour(240, 240, 240)); dc.SetPen (p); dc.SetBrush (b); - dc.DrawRectangle (0, _out_size.height, _panel_size.width, _panel_size.height - _out_size.height); + int const gap = (_panel_size.height - _out_size.height) / 2; + dc.DrawRectangle (0, 0, _panel_size.width, gap); + dc.DrawRectangle (0, gap + _out_size.height + 1, _panel_size.width, gap); } - if (_outline_content && _outline_content->GetValue ()) { + if (_outline_content) { wxPen p (wxColour (255, 0, 0), 2); dc.SetPen (p); dc.SetBrush (*wxTRANSPARENT_BRUSH); - dc.DrawRectangle (_inter_position.x, _inter_position.y, _inter_size.width, _inter_size.height); + dc.DrawRectangle (_inter_position.x, _inter_position.y + (_panel_size.height - _out_size.height) / 2, _inter_size.width, _inter_size.height); } } -/** @param page true if this was a PAGEUP/PAGEDOWN event for which we won't receive a THUMBRELEASE */ void -FilmViewer::slider_moved (bool page) +FilmViewer::set_outline_content (bool o) { - if (!_film) { - return; - } - - if (!page && !_slider_being_moved) { - /* This is the first event of a drag; stop playback for the duration of the drag */ - _was_running_before_slider = stop (); - _slider_being_moved = true; - } - - DCPTime t (_slider->GetValue() * _film->length().get() / 4096); - t = t.round (_film->video_frame_rate()); - /* Ensure that we hit the end of the film at the end of the slider */ - if (t >= _film->length ()) { - t = _film->length() - one_video_frame(); - } - seek (t, false); - update_position_label (); + _outline_content = o; + refresh_panel (); } void -FilmViewer::slider_released () +FilmViewer::set_eyes (Eyes e) { - if (_was_running_before_slider) { - /* Restart after a drag */ - start (); - } - _slider_being_moved = false; + _eyes = e; + slow_refresh (); } void @@ -462,8 +428,7 @@ FilmViewer::panel_sized (wxSizeEvent& ev) if (!quick_refresh()) { slow_refresh (); } - update_position_label (); - update_position_slider (); + PositionChanged (); } void @@ -496,29 +461,15 @@ FilmViewer::calculate_sizes () } void -FilmViewer::play_clicked () -{ - check_play_state (); -} - -void -FilmViewer::check_play_state () +FilmViewer::start () { - if (!_film || _film->video_frame_rate() == 0) { + if (!_film) { return; } - if (_play_button->GetValue()) { - start (); - } else { - stop (); - } -} - -void -FilmViewer::start () -{ - if (!_film) { + optional v = PlaybackPermitted (); + if (v && !*v) { + /* Computer says no */ return; } @@ -530,7 +481,7 @@ FilmViewer::start () _playing = true; _dropped = 0; timer (); - _play_button->SetValue (true); + Started (position()); } bool @@ -546,127 +497,14 @@ FilmViewer::stop () } _playing = false; - _play_button->SetValue (false); + Stopped (position()); return true; } void -FilmViewer::update_position_slider () -{ - if (!_film) { - _slider->SetValue (0); - return; - } - - DCPTime const len = _film->length (); - - if (len.get ()) { - int const new_slider_position = 4096 * _video_position.get() / len.get(); - if (new_slider_position != _slider->GetValue()) { - _slider->SetValue (new_slider_position); - } - } -} - -void -FilmViewer::update_position_label () -{ - if (!_film) { - _frame_number->SetLabel ("0"); - _timecode->SetLabel ("0:0:0.0"); - return; - } - - double const fps = _film->video_frame_rate (); - /* Count frame number from 1 ... not sure if this is the best idea */ - _frame_number->SetLabel (wxString::Format (wxT("%ld"), lrint (_video_position.seconds() * fps) + 1)); - _timecode->SetLabel (time_to_timecode (_video_position, fps)); -} - -void -FilmViewer::active_jobs_changed (optional j) -{ - /* examine content is the only job which stops the viewer working */ - bool const a = !j || *j != "examine_content"; - _slider->Enable (a); - _play_button->Enable (a); -} - -DCPTime -FilmViewer::nudge_amount (wxKeyboardState& ev) -{ - DCPTime amount = one_video_frame (); - - if (ev.ShiftDown() && !ev.ControlDown()) { - amount = DCPTime::from_seconds (1); - } else if (!ev.ShiftDown() && ev.ControlDown()) { - amount = DCPTime::from_seconds (10); - } else if (ev.ShiftDown() && ev.ControlDown()) { - amount = DCPTime::from_seconds (60); - } - - return amount; -} - -void -FilmViewer::go_to (DCPTime t) -{ - if (t < DCPTime ()) { - t = DCPTime (); - } - - if (t >= _film->length ()) { - t = _film->length (); - } - - seek (t, true); - update_position_label (); - update_position_slider (); -} - -void -FilmViewer::rewind_clicked (wxMouseEvent& ev) -{ - go_to(DCPTime()); - ev.Skip(); -} - -void -FilmViewer::back_frame () -{ - if (!_film) { - return; - } - - go_to (_video_position - one_video_frame()); -} - -void -FilmViewer::forward_frame () +FilmViewer::player_change (ChangeType type, int property, bool frequent) { - if (!_film) { - return; - } - - go_to (_video_position + one_video_frame()); -} - -void -FilmViewer::back_clicked (wxKeyboardState& ev) -{ - go_to (_video_position - nudge_amount (ev)); -} - -void -FilmViewer::forward_clicked (wxKeyboardState& ev) -{ - go_to (_video_position + nudge_amount (ev)); -} - -void -FilmViewer::player_changed (int property, bool frequent) -{ - if (frequent) { + if (type != CHANGE_TYPE_DONE || frequent) { return; } @@ -692,38 +530,13 @@ FilmViewer::player_changed (int property, bool frequent) if (!refreshed) { slow_refresh (); } - update_position_label (); - update_position_slider (); + PositionChanged (); } void -FilmViewer::setup_sensitivity () +FilmViewer::film_change (ChangeType type, Film::Property p) { - bool const c = _film && !_film->content().empty (); - - _slider->Enable (c); - _rewind_button->Enable (c); - _back_button->Enable (c); - _forward_button->Enable (c); - _play_button->Enable (c); - if (_outline_content) { - _outline_content->Enable (c); - } - _frame_number->Enable (c); - _timecode->Enable (c); - if (_jump_to_selected) { - _jump_to_selected->Enable (c); - } - - _eye->Enable (c && _film->three_d ()); -} - -void -FilmViewer::film_changed (Film::Property p) -{ - if (p == Film::CONTENT || p == Film::THREE_D) { - setup_sensitivity (); - } else if (p == Film::AUDIO_CHANNELS) { + if (type == CHANGE_TYPE_DONE && p == Film::AUDIO_CHANNELS) { recreate_butler (); } } @@ -746,7 +559,7 @@ FilmViewer::quick_refresh () return false; } - if (!_player_video.first->reset_metadata (_player->video_container_size(), _film->frame_size())) { + if (!_player_video.first->reset_metadata (_film, _player->video_container_size(), _film->frame_size())) { return false; } @@ -755,18 +568,12 @@ FilmViewer::quick_refresh () } void -FilmViewer::set_position (DCPTime p) -{ - _video_position = p; - seek (p, true); - update_position_label (); - update_position_slider (); -} - -void -FilmViewer::set_position (shared_ptr content, ContentTime t) +FilmViewer::seek (shared_ptr content, ContentTime t, bool accurate) { - set_position (_player->content_time_to_dcp (content, t)); + optional dt = _player->content_time_to_dcp (content, t); + if (dt) { + seek (*dt, accurate); + } } void @@ -776,43 +583,25 @@ FilmViewer::set_coalesce_player_changes (bool c) if (!c) { BOOST_FOREACH (int i, _pending_player_changes) { - player_changed (i, false); + player_change (CHANGE_TYPE_DONE, i, false); } _pending_player_changes.clear (); } } void -FilmViewer::timecode_clicked () +FilmViewer::seek (DCPTime t, bool accurate) { - PlayheadToTimecodeDialog* dialog = new PlayheadToTimecodeDialog (this, _film->video_frame_rate ()); - if (dialog->ShowModal() == wxID_OK) { - go_to (dialog->get ()); + if (!_butler) { + return; } - dialog->Destroy (); -} -void -FilmViewer::frame_number_clicked () -{ - PlayheadToFrameDialog* dialog = new PlayheadToFrameDialog (this, _film->video_frame_rate ()); - if (dialog->ShowModal() == wxID_OK) { - go_to (dialog->get ()); + if (t < DCPTime ()) { + t = DCPTime (); } - dialog->Destroy (); -} - -void -FilmViewer::jump_to_selected_clicked () -{ - Config::instance()->set_jump_to_selected (_jump_to_selected->GetValue ()); -} -void -FilmViewer::seek (DCPTime t, bool accurate) -{ - if (!_butler) { - return; + if (t >= _film->length ()) { + t = _film->length (); } bool const was_running = stop (); @@ -824,11 +613,20 @@ FilmViewer::seek (DCPTime t, bool accurate) if (was_running) { start (); } + + PositionChanged (); } void FilmViewer::config_changed (Config::Property p) { +#ifdef DCPOMATIC_VARIANT_SWAROOP + if (p == Config::PLAYER_BACKGROUND_IMAGE) { + refresh_panel (); + return; + } +#endif + if (p != Config::SOUND && p != Config::SOUND_OUTPUT) { return; } @@ -867,7 +665,7 @@ FilmViewer::config_changed (Config::Property p) } catch (RtAudioError& e) { #endif error_dialog ( - this, + _panel, _("Could not set up audio output. There will be no audio during the preview."), std_to_wx(e.what()) ); } @@ -879,6 +677,16 @@ FilmViewer::config_changed (Config::Property p) } } +DCPTime +FilmViewer::uncorrected_time () const +{ + if (_audio.isStreamRunning ()) { + return DCPTime::from_seconds (const_cast(&_audio)->getStreamTime()); + } + + return _video_position; +} + DCPTime FilmViewer::time () const { @@ -893,7 +701,14 @@ FilmViewer::time () const int FilmViewer::audio_callback (void* out_p, unsigned int frames) { - _butler->get_audio (reinterpret_cast (out_p), frames); + while (true) { + optional t = _butler->get_audio (reinterpret_cast (out_p), frames); + if (!t || DCPTime(uncorrected_time() - *t) < one_video_frame()) { + /* There was an underrun or this audio is on time; carry on */ + break; + } + /* The audio we just got was (very) late; drop it and get some more. */ + } boost::mutex::scoped_lock lm (_latency_history_mutex, boost::try_to_lock); if (lm) { @@ -943,6 +758,7 @@ FilmViewer::one_video_frame () const return DCPTime::from_frames (1, _film->video_frame_rate()); } +/** Open a dialog box showing our film's closed captions */ void FilmViewer::show_closed_captions () { @@ -950,9 +766,13 @@ FilmViewer::show_closed_captions () } void -FilmViewer::caption (PlayerCaption c, CaptionType t, DCPTimePeriod p) +FilmViewer::seek_by (DCPTime by, bool accurate) { - if (t == CAPTION_CLOSED) { - _closed_captions_dialog->caption (c, p); - } + seek (_video_position + by, accurate); +} + +void +FilmViewer::set_pad_black (bool p) +{ + _pad_black = p; }