C++11 tidying.

[dcpomatic.git] / src / wx / video_view.cc

diff --git a/src/wx/video_view.cc b/src/wx/video_view.cc
index 7805a1fb3e9204085932b9db4f43a507c279121b..5f44c37d61779db3cced384221af5fdba9289506 100644 (file)
--- a/src/wx/video_view.cc
+++ b/src/wx/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.
 
@@ -18,6 +18,7 @@
 
 */
 
+
 #include "video_view.h"
 #include "wx_util.h"
 #include "film_viewer.h"
@@ -26,22 +27,18 @@
 #include <boost/optional.hpp>
 
 using std::pair;
-using boost::shared_ptr;
+using std::shared_ptr;
 using boost::optional;
 
+
 VideoView::VideoView (FilmViewer* viewer)
        : _viewer (viewer)
        , _state_timer ("viewer")
-       , _video_frame_rate (0)
-       , _eyes (EYES_LEFT)
-       , _three_d (false)
-       , _dropped (0)
-       , _errored (0)
-       , _gets (0)
 {
 
 }
 
+
 void
 VideoView::clear ()
 {
@@ -50,6 +47,7 @@ VideoView::clear ()
        _player_video.second = dcpomatic::DCPTime ();
 }
 
+
 /** Could be called from any thread.
  *  @param non_blocking true to return false quickly if no video is available quickly.
  *  @return FAIL if there's no frame, AGAIN if the method should be called again, or SUCCESS
@@ -62,7 +60,7 @@ VideoView::get_next_frame (bool non_blocking)
                return FAIL;
        }
 
-       shared_ptr<Butler> butler = _viewer->butler ();
+       auto butler = _viewer->butler ();
        if (!butler) {
                return FAIL;
        }
@@ -72,7 +70,7 @@ VideoView::get_next_frame (bool non_blocking)
 
        do {
                Butler::Error e;
-               pair<shared_ptr<PlayerVideo>, dcpomatic::DCPTime> pv = butler->get_video (!non_blocking, &e);
+               auto pv = butler->get_video (!non_blocking, &e);
                if (e.code == Butler::Error::DIED) {
                        LOG_ERROR ("Butler died with %1", e.summary());
                }
@@ -84,7 +82,7 @@ VideoView::get_next_frame (bool non_blocking)
                _player_video.first &&
                _three_d &&
                _eyes != _player_video.first->eyes() &&
-               _player_video.first->eyes() != EYES_BOTH
+               _player_video.first->eyes() != Eyes::BOTH
                );
 
        if (_player_video.first && _player_video.first->error()) {
@@ -94,12 +92,14 @@ VideoView::get_next_frame (bool non_blocking)
        return SUCCESS;
 }
 
+
 dcpomatic::DCPTime
 VideoView::one_video_frame () const
 {
        return dcpomatic::DCPTime::from_frames (1, video_frame_rate());
 }
 
+
 /** @return Time in ms until the next frame is due, or empty if nothing is due */
 optional<int>
 VideoView::time_until_next_frame () const
@@ -109,14 +109,15 @@ VideoView::time_until_next_frame () const
                return optional<int>();
        }
 
-       dcpomatic::DCPTime const next = position() + one_video_frame();
-       dcpomatic::DCPTime const time = _viewer->audio_time().get_value_or(position());
+       auto const next = position() + one_video_frame();
+       auto const time = _viewer->audio_time().get_value_or(position());
        if (next < time) {
                return 0;
        }
        return (next.seconds() - time.seconds()) * 1000;
 }
 
+
 void
 VideoView::start ()
 {
@@ -125,10 +126,11 @@ VideoView::start ()
        _errored = 0;
 }
 
+
 bool
 VideoView::reset_metadata (shared_ptr<const Film> film, dcp::Size player_video_container_size)
 {
-       pair<shared_ptr<PlayerVideo>, dcpomatic::DCPTime> pv = player_video ();
+       auto pv = player_video ();
        if (!pv.first) {
                return false;
        }