projects
/
dcpomatic.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Replace aligned bool with enum Alignment.
[dcpomatic.git]
/
src
/
wx
/
simple_video_view.cc
diff --git
a/src/wx/simple_video_view.cc
b/src/wx/simple_video_view.cc
index 117eee2181241768990be4adecb5c24ca5c03c49..e54c8390e78ea9cd56483c159c214c21a16d536d 100644
(file)
--- a/
src/wx/simple_video_view.cc
+++ b/
src/wx/simple_video_view.cc
@@
-1,5
+1,5
@@
/*
/*
- Copyright (C) 2019 Carl Hetherington <cth@carlh.net>
+ Copyright (C) 2019
-2021
Carl Hetherington <cth@carlh.net>
This file is part of DCP-o-matic.
This file is part of DCP-o-matic.
@@
-18,21
+18,23
@@
*/
*/
-#include "simple_video_view.h"
+
+#include "closed_captions_dialog.h"
#include "film_viewer.h"
#include "film_viewer.h"
+#include "simple_video_view.h"
#include "wx_util.h"
#include "wx_util.h"
-#include "closed_captions_dialog.h"
-#include "lib/image.h"
-#include "lib/dcpomatic_log.h"
#include "lib/butler.h"
#include "lib/butler.h"
+#include "lib/dcpomatic_log.h"
+#include "lib/image.h"
#include <dcp/util.h>
#include <wx/wx.h>
#include <dcp/util.h>
#include <wx/wx.h>
-#include <boost/bind.hpp>
+#include <boost/bind/bind.hpp>
+
using std::max;
using std::max;
+using std::shared_ptr;
using std::string;
using boost::optional;
using std::string;
using boost::optional;
-using boost::shared_ptr;
#if BOOST_VERSION >= 106100
using namespace boost::placeholders;
#endif
#if BOOST_VERSION >= 106100
using namespace boost::placeholders;
#endif
@@
-57,36
+59,38
@@
SimpleVideoView::SimpleVideoView (FilmViewer* viewer, wxWindow* parent)
_timer.Bind (wxEVT_TIMER, boost::bind(&SimpleVideoView::timer, this));
}
_timer.Bind (wxEVT_TIMER, boost::bind(&SimpleVideoView::timer, this));
}
+
void
SimpleVideoView::paint ()
{
_state_timer.set("paint-panel");
wxPaintDC dc (_panel);
void
SimpleVideoView::paint ()
{
_state_timer.set("paint-panel");
wxPaintDC dc (_panel);
- dcp::Size const out_size = _viewer->out_size ();
- wxSize const panel_size = _panel->GetSize ();
+ auto const panel_size = _panel->GetSize ();
- if (!out_size.width || !out_size.height || !_image || out_size != _image->size()) {
+ dcp::Size out_size;
+ if (!_image) {
dc.Clear ();
} else {
dc.Clear ();
} else {
-
+ out_size = _image->size();
wxImage frame (out_size.width, out_size.height, _image->data()[0], true);
wxBitmap frame_bitmap (frame);
dc.DrawBitmap (frame_bitmap, 0, max(0, (panel_size.GetHeight() - out_size.height) / 2));
}
wxImage frame (out_size.width, out_size.height, _image->data()[0], true);
wxBitmap frame_bitmap (frame);
dc.DrawBitmap (frame_bitmap, 0, max(0, (panel_size.GetHeight() - out_size.height) / 2));
}
+ auto pad = pad_colour();
+
if (out_size.width < panel_size.GetWidth()) {
if (out_size.width < panel_size.GetWidth()) {
- /* XXX: these colours are right for GNOME; may need adjusting for other OS */
- wxPen p (_viewer->pad_black() ? wxColour(0, 0, 0) : wxColour(240, 240, 240));
- wxBrush b (_viewer->pad_black() ? wxColour(0, 0, 0) : wxColour(240, 240, 240));
+ wxPen p (pad);
+ wxBrush b (pad);
dc.SetPen (p);
dc.SetBrush (b);
dc.DrawRectangle (out_size.width, 0, panel_size.GetWidth() - out_size.width, panel_size.GetHeight());
}
if (out_size.height < panel_size.GetHeight()) {
dc.SetPen (p);
dc.SetBrush (b);
dc.DrawRectangle (out_size.width, 0, panel_size.GetWidth() - out_size.width, panel_size.GetHeight());
}
if (out_size.height < panel_size.GetHeight()) {
- wxPen p (
_viewer->pad_black() ? wxColour(0, 0, 0) : wxColour(240, 240, 240)
);
- wxBrush b (
_viewer->pad_black() ? wxColour(0, 0, 0) : wxColour(240, 240, 240)
);
+ wxPen p (
pad
);
+ wxBrush b (
pad
);
dc.SetPen (p);
dc.SetBrush (b);
int const gap = (panel_size.GetHeight() - out_size.height) / 2;
dc.SetPen (p);
dc.SetBrush (b);
int const gap = (panel_size.GetHeight() - out_size.height) / 2;
@@
-95,15
+99,15
@@
SimpleVideoView::paint ()
}
if (_viewer->outline_content()) {
}
if (_viewer->outline_content()) {
- wxPen p (
wxColour (255, 0, 0
), 2);
+ wxPen p (
outline_content_colour(
), 2);
dc.SetPen (p);
dc.SetBrush (*wxTRANSPARENT_BRUSH);
dc.DrawRectangle (_inter_position.x, _inter_position.y + (panel_size.GetHeight() - out_size.height) / 2, _inter_size.width, _inter_size.height);
}
dc.SetPen (p);
dc.SetBrush (*wxTRANSPARENT_BRUSH);
dc.DrawRectangle (_inter_position.x, _inter_position.y + (panel_size.GetHeight() - out_size.height) / 2, _inter_size.width, _inter_size.height);
}
-
optional<dcpomatic::Rect<double> >
subs = _viewer->outline_subtitles();
+
auto
subs = _viewer->outline_subtitles();
if (subs) {
if (subs) {
- wxPen p (
wxColour(0, 255, 0
), 2);
+ wxPen p (
outline_subtitles_colour(
), 2);
dc.SetPen (p);
dc.SetBrush (*wxTRANSPARENT_BRUSH);
dc.DrawRectangle (subs->x * out_size.width, subs->y * out_size.height, subs->width * out_size.width, subs->height * out_size.height);
dc.SetPen (p);
dc.SetBrush (*wxTRANSPARENT_BRUSH);
dc.DrawRectangle (subs->x * out_size.width, subs->y * out_size.height, subs->width * out_size.width, subs->height * out_size.height);
@@
-112,6
+116,7
@@
SimpleVideoView::paint ()
_state_timer.unset();
}
_state_timer.unset();
}
+
void
SimpleVideoView::refresh_panel ()
{
void
SimpleVideoView::refresh_panel ()
{
@@
-121,6
+126,7
@@
SimpleVideoView::refresh_panel ()
_state_timer.unset ();
}
_state_timer.unset ();
}
+
void
SimpleVideoView::timer ()
{
void
SimpleVideoView::timer ()
{
@@
-129,7
+135,7
@@
SimpleVideoView::timer ()
}
display_next_frame (false);
}
display_next_frame (false);
-
DCPTime
const next = position() + _viewer->one_video_frame();
+
auto
const next = position() + _viewer->one_video_frame();
if (next >= length()) {
_viewer->finished ();
if (next >= length()) {
_viewer->finished ();
@@
-144,6
+150,7
@@
SimpleVideoView::timer ()
}
}
}
}
+
void
SimpleVideoView::start ()
{
void
SimpleVideoView::start ()
{
@@
-151,24
+158,18
@@
SimpleVideoView::start ()
timer ();
}
timer ();
}
+
/** Try to get a frame from the butler and display it.
* @param non_blocking true to return false quickly if no video is available quickly (i.e. we are waiting for the butler).
* false to ask the butler to block until it has video (unless it is suspended).
* @return true on success, false if we did nothing because it would have taken too long.
*/
/** Try to get a frame from the butler and display it.
* @param non_blocking true to return false quickly if no video is available quickly (i.e. we are waiting for the butler).
* false to ask the butler to block until it has video (unless it is suspended).
* @return true on success, false if we did nothing because it would have taken too long.
*/
-bool
+VideoView::NextFrameResult
SimpleVideoView::display_next_frame (bool non_blocking)
{
SimpleVideoView::display_next_frame (bool non_blocking)
{
- bool r = get_next_frame (non_blocking);
- if (!r) {
- if (non_blocking) {
- /* No video available; return saying we failed */
- return false;
- } else {
- /* Player was suspended; come back later */
- signal_manager->when_idle (boost::bind(&SimpleVideoView::display_next_frame, this, false));
- return false;
- }
+ auto const r = get_next_frame (non_blocking);
+ if (r != SUCCESS) {
+ return r;
}
update ();
}
update ();
@@
-179,9
+180,10
@@
SimpleVideoView::display_next_frame (bool non_blocking)
error_dialog (get(), e.what());
}
error_dialog (get(), e.what());
}
- return
true
;
+ return
SUCCESS
;
}
}
+
void
SimpleVideoView::update ()
{
void
SimpleVideoView::update ()
{
@@
-220,7
+222,7
@@
SimpleVideoView::update ()
_state_timer.set ("get image");
set_image (
_state_timer.set ("get image");
set_image (
- player_video().first->image(bind(&PlayerVideo::force, _1, AV_PIX_FMT_RGB24), V
IDEO_RANGE_FULL, false
, true)
+ player_video().first->image(bind(&PlayerVideo::force, _1, AV_PIX_FMT_RGB24), V
ideoRange::FULL, Image::Alignment::COMPACT
, true)
);
_state_timer.set ("ImageChanged");
);
_state_timer.set ("ImageChanged");