X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fwx%2Fvideo_view.h;h=d7d60c21da14b4e933af418a8ea79ad223fdc574;hb=670acb60e8b0c78b66ea60657a3586b308fe7c93;hp=e5fbb671dd20e7f770fee744a5c8c676fd45d509;hpb=6516fb170b0e7a83c582a858fb54d1f83f751cc0;p=dcpomatic.git diff --git a/src/wx/video_view.h b/src/wx/video_view.h index e5fbb671d..d7d60c21d 100644 --- a/src/wx/video_view.h +++ b/src/wx/video_view.h @@ -1,5 +1,5 @@ /* - Copyright (C) 2019 Carl Hetherington + Copyright (C) 2019-2021 Carl Hetherington This file is part of DCP-o-matic. @@ -18,17 +18,19 @@ */ + #ifndef DCPOMATIC_VIDEO_VIEW_H #define DCPOMATIC_VIDEO_VIEW_H + #include "lib/dcpomatic_time.h" +#include "lib/exception_store.h" +#include "lib/signaller.h" #include "lib/timer.h" #include "lib/types.h" -#include "lib/exception_store.h" -#include #include #include -#include + class Image; class wxWindow; @@ -36,12 +38,16 @@ class FilmViewer; class Player; class PlayerVideo; -class VideoView : public ExceptionStore, public boost::noncopyable + +class VideoView : public ExceptionStore, public Signaller { public: VideoView (FilmViewer* viewer); virtual ~VideoView () {} + VideoView (VideoView const&) = delete; + VideoView& operator= (VideoView const&) = delete; + /** @return the thing displaying the image */ virtual wxWindow* get () const = 0; /** Re-make and display the image from the current _player_video */ @@ -61,10 +67,12 @@ public: virtual NextFrameResult display_next_frame (bool) = 0; void clear (); - bool reset_metadata (boost::shared_ptr film, dcp::Size player_video_container_size); + bool reset_metadata (std::shared_ptr film, dcp::Size player_video_container_size); /** Emitted from the GUI thread when our display changes in size */ boost::signals2::signal Sized; + /** Emitted from the GUI thread when a lot of frames are being dropped */ + boost::signals2::signal TooManyDropped; /* Accessors for FilmViewer */ @@ -133,15 +141,12 @@ protected: return _length; } - std::pair, dcpomatic::DCPTime> player_video () const { + std::pair, dcpomatic::DCPTime> player_video () const { boost::mutex::scoped_lock lm (_mutex); return _player_video; } - void add_dropped () { - boost::mutex::scoped_lock lm (_mutex); - ++_dropped; - } + void add_dropped (); void add_get () { boost::mutex::scoped_lock lm (_mutex); @@ -156,16 +161,18 @@ private: /** Mutex protecting all the state in this class */ mutable boost::mutex _mutex; - std::pair, dcpomatic::DCPTime> _player_video; - int _video_frame_rate; + std::pair, dcpomatic::DCPTime> _player_video; + int _video_frame_rate = 0; /** length of the film we are playing, or 0 if there is none */ dcpomatic::DCPTime _length; - Eyes _eyes; - bool _three_d; + Eyes _eyes = Eyes::LEFT; + bool _three_d = false; - int _dropped; - int _errored; - int _gets; + int _dropped = 0; + struct timeval _dropped_check_period_start; + int _errored = 0; + int _gets = 0; }; + #endif