X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Flib%2Fbutler.h;h=8c4d30efefbad224ce47731c353791511296160d;hb=94ee305348f7d3eb548fd44ca4aa1c57645056b1;hp=2cb49c1a6c2ab49fd6cf3961e97fe5544f77b26e;hpb=d8dd8b76a5b0f59e11392ab261f2410102bfe934;p=dcpomatic.git diff --git a/src/lib/butler.h b/src/lib/butler.h index 2cb49c1a6..8c4d30efe 100644 --- a/src/lib/butler.h +++ b/src/lib/butler.h @@ -20,6 +20,7 @@ #include "video_ring_buffers.h" #include "audio_ring_buffers.h" +#include "text_ring_buffers.h" #include "audio_mapping.h" #include "exception_store.h" #include @@ -27,53 +28,94 @@ #include #include #include +#include -class Film; class Player; class PlayerVideo; class Butler : public ExceptionStore, public boost::noncopyable { public: - Butler (boost::weak_ptr film, boost::shared_ptr player, AudioMapping map, int audio_channels); + Butler ( + boost::shared_ptr player, + AudioMapping map, + int audio_channels, + boost::function pixel_format, + bool aligned, + bool fast + ); + ~Butler (); void seek (DCPTime position, bool accurate); - std::pair, DCPTime> get_video (); - void get_audio (float* out, Frame frames); + + enum Error { + NONE, + AGAIN + }; + + std::pair, DCPTime> get_video (Error* e = 0); + boost::optional get_audio (float* out, Frame frames); + boost::optional get_closed_caption (); void disable_audio (); + std::pair memory_used () const; + private: void thread (); void video (boost::shared_ptr video, DCPTime time); - void audio (boost::shared_ptr audio, DCPTime time); - void player_changed (); + void audio (boost::shared_ptr audio, DCPTime time, int frame_rate); + void text (PlayerText pt, TextType type, boost::optional track, DCPTimePeriod period); bool should_run () const; + void prepare (boost::weak_ptr video); + void player_change (ChangeType type, bool frequent); + void seek_unlocked (DCPTime position, bool accurate); - boost::weak_ptr _film; boost::shared_ptr _player; boost::thread* _thread; + /** mutex to protect _video, _audio and _closed_caption for when we are clearing them and they all need to be + cleared together without any data being inserted in the interim; + XXX: is this necessary now that all butler output data is timestamped? Perhaps the locked clear-out + is only required if we guarantee that get_video() and get_audio() calls are in sync. + */ + boost::mutex _buffers_mutex; VideoRingBuffers _video; AudioRingBuffers _audio; + TextRingBuffers _closed_caption; + + boost::thread_group _prepare_pool; + boost::asio::io_service _prepare_service; + boost::shared_ptr _prepare_work; + /** mutex to protect _pending_seek_position, _pending_seek_acurate, _finished, _died, _stop_thread */ boost::mutex _mutex; boost::condition _summon; boost::condition _arrived; boost::optional _pending_seek_position; bool _pending_seek_accurate; - + int _suspended; bool _finished; bool _died; + bool _stop_thread; AudioMapping _audio_mapping; int _audio_channels; - bool _stop_thread; bool _disable_audio; + boost::function _pixel_format; + bool _aligned; + bool _fast; + + /** If we are waiting to be refilled following a seek, this is the time we were + seeking to. + */ + boost::optional _awaiting; + boost::signals2::scoped_connection _player_video_connection; boost::signals2::scoped_connection _player_audio_connection; - boost::signals2::scoped_connection _player_changed_connection; + boost::signals2::scoped_connection _player_text_connection; + boost::signals2::scoped_connection _player_change_connection; };