*/
-#include "markers_dialog.h"
-#include "wx_util.h"
-#include "timecode.h"
-#include "static_text.h"
-#include "dcpomatic_button.h"
#include "check_box.h"
+#include "dcpomatic_button.h"
#include "film_viewer.h"
+#include "markers.h"
+#include "markers_dialog.h"
+#include "static_text.h"
+#include "timecode.h"
+#include "wx_util.h"
#include "lib/film.h"
#include <dcp/types.h>
+#include <dcp/warnings.h>
+LIBDCP_DISABLE_WARNINGS
#include <wx/gbsizer.h>
+LIBDCP_ENABLE_WARNINGS
#include <boost/bind/bind.hpp>
-#include <iostream>
-using std::cout;
+using std::make_shared;
using std::shared_ptr;
using std::weak_ptr;
-using std::make_shared;
-using boost::optional;
using boost::bind;
+using boost::optional;
using dcpomatic::DCPTime;
class Marker
{
public:
- Marker (wxWindow* parent, wxGridBagSizer* grid, int row, weak_ptr<Film> film_, weak_ptr<FilmViewer> viewer_, wxString name, dcp::Marker type_)
- : film (film_)
- , viewer (viewer_)
- , type (type_)
+ Marker(wxWindow* parent, wxGridBagSizer* grid, int row, weak_ptr<Film> film, FilmViewer const& viewer, wxString name, dcp::Marker type)
+ : _film(film)
+ , _viewer(viewer)
+ , _type(type)
{
- checkbox = new CheckBox(parent, name);
- grid->Add (checkbox, wxGBPosition(row, 0), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
- timecode = new Timecode<DCPTime> (parent);
- grid->Add (timecode, wxGBPosition(row, 1));
- set_button = new Button (parent, _("Set from current position"));
- grid->Add (set_button, wxGBPosition(row, 2));
-
- auto f = film.lock ();
+ _checkbox = new CheckBox(parent, name);
+ grid->Add(_checkbox, wxGBPosition(row, 0), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
+ _timecode = new Timecode<DCPTime>(parent);
+ grid->Add(_timecode, wxGBPosition(row, 1));
+ _set_button = new Button(parent, _("Set from current position"));
+ grid->Add(_set_button, wxGBPosition(row, 2));
+
+ auto f = _film.lock();
DCPOMATIC_ASSERT (f);
- auto t = f->marker (type);
- checkbox->SetValue (static_cast<bool>(t));
+ auto t = f->marker(_type);
+ _checkbox->SetValue(static_cast<bool>(t));
if (t) {
- timecode->set (*t, f->video_frame_rate());
+ _timecode->set(*t, f->video_frame_rate());
}
set_sensitivity ();
- set_button->Bind (wxEVT_BUTTON, bind(&Marker::set, this));
- checkbox->Bind (wxEVT_CHECKBOX, bind(&Marker::checkbox_clicked, this));
- timecode->Changed.connect (bind(&Marker::changed, this));
+ _set_button->Bind(wxEVT_BUTTON, bind(&Marker::set, this));
+ _checkbox->bind(&Marker::checkbox_clicked, this);
+ _timecode->Changed.connect(bind(&Marker::changed, this));
}
private:
void set_sensitivity ()
{
- timecode->Enable (checkbox->GetValue());
- set_button->Enable (checkbox->GetValue());
+ _timecode->Enable(_checkbox->GetValue());
+ _set_button->Enable(_checkbox->GetValue());
}
void set ()
{
- auto f = film.lock ();
+ auto f = _film.lock();
DCPOMATIC_ASSERT (f);
- auto v = viewer.lock ();
- DCPOMATIC_ASSERT (v);
- timecode->set (v->position(), f->video_frame_rate());
+ _timecode->set(_viewer.position(), f->video_frame_rate());
changed ();
}
void changed ()
{
- auto f = film.lock ();
+ auto f = _film.lock();
DCPOMATIC_ASSERT (f);
auto vfr = f->video_frame_rate();
- auto tc = timecode->get(vfr);
+ auto tc = _timecode->get(vfr);
if (tc >= f->length()) {
tc = f->length() - DCPTime::from_frames(1, vfr);
- timecode->set (tc, vfr);
+ _timecode->set(tc, vfr);
}
- if (checkbox->GetValue()) {
- f->set_marker (type, tc);
+ if (_checkbox->GetValue()) {
+ f->set_marker(_type, tc);
} else {
- f->unset_marker (type);
+ f->unset_marker(_type);
}
}
- weak_ptr<Film> film;
- weak_ptr<FilmViewer> viewer;
- dcp::Marker type;
- CheckBox* checkbox;
- Timecode<dcpomatic::DCPTime>* timecode;
- Button* set_button;
+ weak_ptr<Film> _film;
+ FilmViewer const& _viewer;
+ dcp::Marker _type;
+ CheckBox* _checkbox;
+ Timecode<dcpomatic::DCPTime>* _timecode;
+ Button* _set_button;
};
-MarkersDialog::MarkersDialog (wxWindow* parent, weak_ptr<Film> film, weak_ptr<FilmViewer> viewer)
+MarkersDialog::MarkersDialog(wxWindow* parent, weak_ptr<Film> film, FilmViewer const& viewer)
: wxDialog (parent, wxID_ANY, _("Markers"))
, _film (film)
{
auto grid = new wxGridBagSizer (DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
int r = 0;
- _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("First frame of composition"), dcp::Marker::FFOC));
- _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("Last frame of composition"), dcp::Marker::LFOC));
- _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("First frame of title credits"), dcp::Marker::FFTC));
- _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("Last frame of title credits"), dcp::Marker::LFTC));
- _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("First frame of intermission"), dcp::Marker::FFOI));
- _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("Last frame of intermission"), dcp::Marker::LFOI));
- _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("First frame of end credits"), dcp::Marker::FFEC));
- _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("Last frame of end credits"), dcp::Marker::LFEC));
- _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("First frame of moving credits"), dcp::Marker::FFMC));
- _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, _("Last frame of moving credits"), dcp::Marker::LFMC));
+ for (auto const& marker: all_editable_markers()) {
+ _markers.push_back (make_shared<Marker>(this, grid, r++, film, viewer, marker.first, marker.second));
+ }
sizer->Add (grid, 0, wxALL, 8);
projects / dcpomatic.git / blobdiff