/*
- Copyright (C) 2018-2019 Carl Hetherington <cth@carlh.net>
+ Copyright (C) 2018-2021 Carl Hetherington <cth@carlh.net>
This file is part of DCP-o-matic.
*/
+
+#include "check_box.h"
+#include "content_view.h"
#include "controls.h"
+#include "dcpomatic_button.h"
#include "film_viewer.h"
-#include "wx_util.h"
-#include "playhead_to_timecode_dialog.h"
+#include "markers_panel.h"
#include "playhead_to_frame_dialog.h"
-#include "content_view.h"
+#include "playhead_to_timecode_dialog.h"
#include "static_text.h"
-#include "check_box.h"
-#include "dcpomatic_button.h"
-#include "lib/job_manager.h"
-#include "lib/player_video.h"
-#include "lib/dcp_content.h"
-#include "lib/job.h"
-#include "lib/examine_content_job.h"
+#include "wx_util.h"
#include "lib/content_factory.h"
#include "lib/cross.h"
-#include <dcp/dcp.h>
+#include "lib/dcp_content.h"
+#include "lib/examine_content_job.h"
+#include "lib/film.h"
+#include "lib/job.h"
+#include "lib/job_manager.h"
+#include "lib/player_video.h"
+#include "lib/scope_guard.h"
#include <dcp/cpl.h>
+#include <dcp/dcp.h>
#include <dcp/reel.h>
#include <dcp/reel_picture_asset.h>
-#include <wx/wx.h>
-#include <wx/tglbtn.h>
+#include <dcp/warnings.h>
+LIBDCP_DISABLE_WARNINGS
#include <wx/listctrl.h>
#include <wx/progdlg.h>
+#include <wx/tglbtn.h>
+#include <wx/wx.h>
+LIBDCP_ENABLE_WARNINGS
+
-using std::string;
-using std::list;
using std::cout;
-using std::make_pair;
+using std::dynamic_pointer_cast;
using std::exception;
+using std::list;
+using std::make_pair;
+using std::shared_ptr;
+using std::string;
+using std::weak_ptr;
using boost::optional;
-using boost::shared_ptr;
-using boost::weak_ptr;
-using boost::dynamic_pointer_cast;
+#if BOOST_VERSION >= 106100
+using namespace boost::placeholders;
+#endif
using namespace dcpomatic;
-Controls::Controls (wxWindow* parent, shared_ptr<FilmViewer> viewer, bool editor_controls)
+
+Controls::Controls(wxWindow* parent, FilmViewer& viewer, bool editor_controls)
: wxPanel (parent)
- , _slider (new wxSlider (this, wxID_ANY, 0, 0, 4096))
+ , _markers (new MarkersPanel(this, viewer))
+ , _slider (new wxSlider(this, wxID_ANY, 0, 0, 4096))
, _viewer (viewer)
- , _slider_being_moved (false)
- , _outline_content (0)
- , _eye (0)
- , _jump_to_selected (0)
- , _rewind_button (new Button (this, wxT("|<")))
- , _back_button (new Button (this, wxT("<")))
- , _forward_button (new Button (this, wxT(">")))
- , _frame_number (new StaticText (this, wxT("")))
- , _timecode (new StaticText (this, wxT("")))
+ , _rewind_button (new Button(this, wxT("|<")))
+ , _back_button (new Button(this, wxT("<")))
+ , _forward_button (new Button(this, wxT(">")))
+ , _frame_number (new StaticText(this, wxT("")))
+ , _timecode (new StaticText(this, wxT("")))
, _timer (this)
{
_v_sizer = new wxBoxSizer (wxVERTICAL);
SetSizer (_v_sizer);
- wxBoxSizer* view_options = new wxBoxSizer (wxHORIZONTAL);
+ auto view_options = new wxBoxSizer (wxHORIZONTAL);
if (editor_controls) {
_outline_content = new CheckBox (this, _("Outline content"));
view_options->Add (_outline_content, 0, wxRIGHT | wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_GAP);
_v_sizer->Add (view_options, 0, wxALL, DCPOMATIC_SIZER_GAP);
- wxBoxSizer* h_sizer = new wxBoxSizer (wxHORIZONTAL);
+ auto h_sizer = new wxBoxSizer (wxHORIZONTAL);
- wxBoxSizer* time_sizer = new wxBoxSizer (wxVERTICAL);
+ auto 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 (time_sizer, 0, wxEXPAND);
- h_sizer->Add (_back_button, 0, wxALL, 2);
- h_sizer->Add (_forward_button, 0, wxALL, 2);
+ h_sizer->Add (_rewind_button, 0, wxALL | wxALIGN_CENTER_VERTICAL, 2);
+ h_sizer->Add (time_sizer, 0, wxALL | wxALIGN_CENTER_VERTICAL, 2);
+ h_sizer->Add (_back_button, 0, wxALL | wxALIGN_CENTER_VERTICAL, 2);
+ h_sizer->Add (_forward_button, 0, wxALL | wxALIGN_CENTER_VERTICAL, 2);
_button_sizer = new wxBoxSizer (wxHORIZONTAL);
h_sizer->Add (_button_sizer, 0, wxEXPAND);
- h_sizer->Add (_slider, 1, wxEXPAND);
+ {
+ auto box = new wxBoxSizer (wxVERTICAL);
+ box->Add (_markers, 0, wxEXPAND);
+ box->Add (_slider, 0, wxEXPAND);
+ h_sizer->Add (box, 1, wxEXPAND);
+ }
_v_sizer->Add (h_sizer, 0, wxEXPAND | wxALL, 6);
+#ifdef __WXGTK3__
+ _frame_number->SetMinSize (wxSize(100, -1));
+ _rewind_button->SetMinSize (wxSize(48, -1));
+ _back_button->SetMinSize (wxSize(48, -1));
+ _forward_button->SetMinSize (wxSize(48, -1));
+#else
_frame_number->SetMinSize (wxSize (84, -1));
_rewind_button->SetMinSize (wxSize (32, -1));
_back_button->SetMinSize (wxSize (32, -1));
_forward_button->SetMinSize (wxSize (32, -1));
+#endif
if (_eye) {
_eye->Bind (wxEVT_CHOICE, boost::bind (&Controls::eye_changed, this));
}
if (_outline_content) {
- _outline_content->Bind (wxEVT_CHECKBOX, boost::bind (&Controls::outline_content_changed, this));
+ _outline_content->bind(&Controls::outline_content_changed, this);
}
_slider->Bind (wxEVT_SCROLL_THUMBTRACK, boost::bind(&Controls::slider_moved, this, false));
_frame_number->Bind (wxEVT_LEFT_DOWN, boost::bind(&Controls::frame_number_clicked, this));
_timecode->Bind (wxEVT_LEFT_DOWN, boost::bind(&Controls::timecode_clicked, this));
if (_jump_to_selected) {
- _jump_to_selected->Bind (wxEVT_CHECKBOX, boost::bind (&Controls::jump_to_selected_clicked, this));
+ _jump_to_selected->bind(&Controls::jump_to_selected_clicked, this);
_jump_to_selected->SetValue (Config::instance()->jump_to_selected ());
}
- _viewer->Started.connect (boost::bind(&Controls::started, this));
- _viewer->Stopped.connect (boost::bind(&Controls::stopped, this));
+ viewer.Started.connect (boost::bind(&Controls::started, this));
+ viewer.Stopped.connect (boost::bind(&Controls::stopped, this));
Bind (wxEVT_TIMER, boost::bind(&Controls::update_position, this));
_timer.Start (80, wxTIMER_CONTINUOUS);
- set_film (_viewer->film());
-
- setup_sensitivity ();
+ set_film(viewer.film());
JobManager::instance()->ActiveJobsChanged.connect (
bind (&Controls::active_jobs_changed, this, _2)
);
_config_changed_connection = Config::instance()->Changed.connect (bind(&Controls::config_changed, this, _1));
- config_changed (Config::OTHER);
}
void
setup_sensitivity ();
}
+
void
Controls::started ()
{
setup_sensitivity ();
}
+
void
Controls::stopped ()
{
setup_sensitivity ();
}
+
void
Controls::update_position ()
{
- if (!_slider_being_moved) {
+ if (!_slider_being_moved && !_viewer.pending_idle_get()) {
update_position_label ();
update_position_slider ();
}
}
+
void
Controls::eye_changed ()
{
- _viewer->set_eyes (_eye->GetSelection() == 0 ? EYES_LEFT : EYES_RIGHT);
+ _viewer.set_eyes(_eye->GetSelection() == 0 ? Eyes::LEFT : Eyes::RIGHT);
}
+
void
Controls::outline_content_changed ()
{
- _viewer->set_outline_content (_outline_content->GetValue());
+ _viewer.set_outline_content(_outline_content->GetValue());
}
+
/** @param page true if this was a PAGEUP/PAGEDOWN event for which we won't receive a THUMBRELEASE */
void
Controls::slider_moved (bool page)
if (!page && !_slider_being_moved) {
/* This is the first event of a drag; stop playback for the duration of the drag */
- _viewer->suspend ();
+ _viewer.suspend();
_slider_being_moved = true;
}
*/
bool accurate = false;
if (t >= _film->length ()) {
- t = _film->length() - _viewer->one_video_frame();
+ t = _film->length() - _viewer.one_video_frame();
accurate = true;
}
- _viewer->seek (t, accurate);
+ _viewer.seek(t, accurate);
update_position_label ();
-
- log (
- wxString::Format(
- "playback-seeked %s", t.timecode(_film->video_frame_rate()).c_str()
- )
- );
}
+
void
Controls::slider_released ()
{
/* Restart after a drag */
- _viewer->resume ();
+ _viewer.resume();
_slider_being_moved = false;
}
+
void
Controls::update_position_slider ()
{
return;
}
- DCPTime const len = _film->length ();
+ auto const len = _film->length ();
if (len.get ()) {
- int const new_slider_position = 4096 * _viewer->position().get() / len.get();
+ int const new_slider_position = 4096 * _viewer.position().get() / len.get();
if (new_slider_position != _slider->GetValue()) {
_slider->SetValue (new_slider_position);
}
}
}
+
void
Controls::update_position_label ()
{
if (!_film) {
- _frame_number->SetLabel ("0");
- _timecode->SetLabel ("0:0:0.0");
+ checked_set (_frame_number, wxT("0"));
+ checked_set (_timecode, wxT("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 (_viewer->position().seconds() * fps) + 1));
- _timecode->SetLabel (time_to_timecode (_viewer->position(), fps));
+ checked_set(_frame_number, wxString::Format (wxT("%ld"), lrint(_viewer.position().seconds() * fps) + 1));
+ checked_set(_timecode, time_to_timecode(_viewer.position(), fps));
}
+
void
Controls::active_jobs_changed (optional<string> j)
{
setup_sensitivity ();
}
+
DCPTime
Controls::nudge_amount (wxKeyboardState& ev)
{
- DCPTime amount = _viewer->one_video_frame ();
+ auto amount = _viewer.one_video_frame();
if (ev.ShiftDown() && !ev.ControlDown()) {
amount = DCPTime::from_seconds (1);
return amount;
}
+
void
Controls::rewind_clicked (wxMouseEvent& ev)
{
- _viewer->seek (DCPTime(), true);
+ _viewer.seek(DCPTime(), true);
ev.Skip();
}
+
void
Controls::back_frame ()
{
- _viewer->seek_by (-_viewer->one_video_frame(), true);
+ _viewer.seek_by(-_viewer.one_video_frame(), true);
}
+
void
Controls::forward_frame ()
{
- _viewer->seek_by (_viewer->one_video_frame(), true);
+ _viewer.seek_by(_viewer.one_video_frame(), true);
}
+
void
Controls::back_clicked (wxKeyboardState& ev)
{
- _viewer->seek_by (-nudge_amount(ev), true);
+ _viewer.seek_by(-nudge_amount(ev), true);
}
+
void
Controls::forward_clicked (wxKeyboardState& ev)
{
- _viewer->seek_by (nudge_amount(ev), true);
+ _viewer.seek_by(nudge_amount(ev), true);
}
+
void
Controls::setup_sensitivity ()
{
}
}
+
void
Controls::timecode_clicked ()
{
- PlayheadToTimecodeDialog* dialog = new PlayheadToTimecodeDialog (this, _film->video_frame_rate ());
+ auto dialog = make_wx<PlayheadToTimecodeDialog>(this, _viewer.position(), _film->video_frame_rate());
+
if (dialog->ShowModal() == wxID_OK) {
- _viewer->seek (dialog->get(), true);
+ _viewer.seek(dialog->get(), true);
}
- dialog->Destroy ();
}
+
void
Controls::frame_number_clicked ()
{
- PlayheadToFrameDialog* dialog = new PlayheadToFrameDialog (this, _film->video_frame_rate ());
+ auto dialog = make_wx<PlayheadToFrameDialog>(this, _viewer.position(), _film->video_frame_rate());
+
if (dialog->ShowModal() == wxID_OK) {
- _viewer->seek (dialog->get(), true);
+ _viewer.seek(dialog->get(), true);
}
- dialog->Destroy ();
}
+
void
Controls::jump_to_selected_clicked ()
{
Config::instance()->set_jump_to_selected (_jump_to_selected->GetValue ());
}
+
void
Controls::set_film (shared_ptr<Film> film)
{
_film = film;
+ _markers->set_film (_film);
+
if (_film) {
_film_change_connection = _film->Change.connect (boost::bind(&Controls::film_change, this, _1, _2));
}
update_position_label ();
}
+
shared_ptr<Film>
Controls::film () const
{
return _film;
}
+
void
-Controls::film_change (ChangeType type, Film::Property p)
+Controls::film_change(ChangeType type, FilmProperty p)
{
- if (type == CHANGE_TYPE_DONE) {
- if (p == Film::CONTENT) {
+ if (type == ChangeType::DONE) {
+ if (p == FilmProperty::CONTENT) {
setup_sensitivity ();
update_position_label ();
update_position_slider ();
- } else if (p == Film::THREE_D) {
+ } else if (p == FilmProperty::THREE_D) {
setup_sensitivity ();
}
}
}
-#ifdef DCPOMATIC_PLAYER_STRESS_TEST
+
void
Controls::seek (int slider)
{
slider_moved (false);
slider_released ();
}
-#endif