X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Flib%2Fbutler.h;h=4f4e508843d8cb04f45fc545a4f13119b8b5ecdd;hb=ad1ef39eda58b3a919ea3b7084401a0439409ec6;hp=948a78ea5f64d79dfa82c9ddc864f48b56556927;hpb=565a0ab9fc8be5c21685b019aa06d37e6c66160c;p=dcpomatic.git

diff --git a/src/lib/butler.h b/src/lib/butler.h
index 948a78ea5..4f4e50884 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 <boost/shared_ptr.hpp>
@@ -27,52 +28,94 @@
 #include <boost/thread.hpp>
 #include <boost/thread/condition.hpp>
 #include <boost/signals2.hpp>
+#include <boost/asio.hpp>
 
-class Film;
 class Player;
 class PlayerVideo;
 
 class Butler : public ExceptionStore, public boost::noncopyable
 {
 public:
-	Butler (boost::weak_ptr<const Film> film, boost::shared_ptr<Player> player, AudioMapping map, int audio_channels);
+	Butler (
+		boost::shared_ptr<Player> player,
+		AudioMapping map,
+		int audio_channels,
+		boost::function<AVPixelFormat (AVPixelFormat)> pixel_format,
+		bool aligned,
+		bool fast
+		);
+
 	~Butler ();
 
 	void seek (DCPTime position, bool accurate);
-	std::pair<boost::shared_ptr<PlayerVideo>, DCPTime> get_video ();
-	void get_audio (float* out, Frame frames);
+
+	enum Error {
+		NONE,
+		AGAIN
+	};
+
+	std::pair<boost::shared_ptr<PlayerVideo>, DCPTime> get_video (Error* e = 0);
+	boost::optional<DCPTime> get_audio (float* out, Frame frames);
+	boost::optional<TextRingBuffers::Data> get_closed_caption ();
 
 	void disable_audio ();
 
+	std::pair<size_t, std::string> memory_used () const;
+
 private:
 	void thread ();
 	void video (boost::shared_ptr<PlayerVideo> video, DCPTime time);
 	void audio (boost::shared_ptr<AudioBuffers> audio, DCPTime time);
-	void player_changed ();
+	void text (PlayerText pt, TextType type, boost::optional<DCPTextTrack> track, DCPTimePeriod period);
 	bool should_run () const;
+	void prepare (boost::weak_ptr<PlayerVideo> video) const;
+	void player_change (ChangeType type, bool frequent);
+	void seek_unlocked (DCPTime position, bool accurate);
 
-	boost::weak_ptr<const Film> _film;
 	boost::shared_ptr<Player> _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<boost::asio::io_service::work> _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<DCPTime> _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<AVPixelFormat (AVPixelFormat)> _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<DCPTime> _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;
 };