Fix missing verification note.

[dcpomatic.git] / src / lib / player.h

diff --git a/src/lib/player.h b/src/lib/player.h
index 48ed568dad99d445b8e871837489eedabdf50347..8fc02d9c676312829ce8fb2ee497bf9c0ebeefd2 100644 (file)
--- a/src/lib/player.h
+++ b/src/lib/player.h
@@ -67,13 +67,16 @@ public:
 /** @class Player
  *  @brief A class which can play a Playlist.
  */
-class Player : public std::enable_shared_from_this<Player>, public boost::noncopyable
+class Player : public std::enable_shared_from_this<Player>
 {
 public:
        Player (std::shared_ptr<const Film>);
        Player (std::shared_ptr<const Film>, std::shared_ptr<const Playlist> playlist);
        ~Player ();
 
+       Player (Player const& Player) = delete;
+       Player& operator= (Player const& Player) = delete;
+
        bool pass ();
        void seek (dcpomatic::DCPTime time, bool accurate);
 
@@ -99,6 +102,9 @@ public:
 
        /** Emitted when a video frame is ready.  These emissions happen in the correct order. */
        boost::signals2::signal<void (std::shared_ptr<PlayerVideo>, dcpomatic::DCPTime)> Video;
+       /** Emitted when audio data is ready.  First parameter is the audio data, second its time,
+        *  third the frame rate.
+        */
        boost::signals2::signal<void (std::shared_ptr<AudioBuffers>, dcpomatic::DCPTime, int)> Audio;
        /** Emitted when a text is ready.  This signal may be emitted considerably
         *  after the corresponding Video.
@@ -116,6 +122,7 @@ private:
        friend struct empty_test1;
        friend struct empty_test2;
        friend struct check_reuse_old_data_test;
+       friend struct overlap_video_test1;
 
        void construct ();
        void setup_pieces ();
@@ -171,17 +178,17 @@ private:
        std::shared_ptr<Image> _black_image;
 
        /** true if the player should ignore all video; i.e. never produce any */
-       bool _ignore_video;
-       bool _ignore_audio;
+       bool _ignore_video = false;
+       bool _ignore_audio = false;
        /** true if the player should ignore all text; i.e. never produce any */
-       bool _ignore_text;
-       bool _always_burn_open_subtitles;
+       bool _ignore_text = false;
+       bool _always_burn_open_subtitles = false;
        /** true if we should try to be fast rather than high quality */
-       bool _fast;
+       bool _fast = false;
        /** true if we should keep going in the face of `survivable' errors */
-       bool _tolerant;
+       bool _tolerant = false;
        /** true if we should `play' (i.e output) referenced DCP data (e.g. for preview) */
-       bool _play_referenced;
+       bool _play_referenced = false;
 
        /** Time just after the last video frame we emitted, or the time of the last accurate seek */
        boost::optional<dcpomatic::DCPTime> _last_video_time;
@@ -195,7 +202,7 @@ private:
        LastVideoMap _last_video;
 
        AudioMerger _audio_merger;
-       Shuffler* _shuffler;
+       Shuffler* _shuffler = nullptr;
        std::list<std::pair<std::shared_ptr<PlayerVideo>, dcpomatic::DCPTime> > _delay;
 
        class StreamState
@@ -216,7 +223,7 @@ private:
        Empty _black;
        Empty _silent;
 
-       ActiveText _active_texts[TEXT_COUNT];
+       ActiveText _active_texts[static_cast<int>(TextType::COUNT)];
        std::shared_ptr<AudioProcessor> _audio_processor;
 
        dcpomatic::DCPTime _playback_length;