#include "butler.h"
#include "player.h"
+#include "util.h"
#include <boost/weak_ptr.hpp>
#include <boost/shared_ptr.hpp>
, _player (player)
, _pending_seek_accurate (false)
, _finished (false)
+ , _died (false)
+ , _stop_thread (false)
, _audio_mapping (audio_mapping)
, _audio_channels (audio_channels)
- , _stop_thread (false)
, _disable_audio (false)
{
_player_video_connection = _player->Video.connect (bind (&Butler::video, this, _1, _2));
- _player_audio_connection = _player->Audio.connect (bind (&Butler::audio, this, _1, _2));
+ _player_audio_connection = _player->Audio.connect (bind (&Butler::audio, this, _1));
_player_changed_connection = _player->Changed.connect (bind (&Butler::player_changed, this));
_thread = new boost::thread (bind (&Butler::thread, this));
}
Butler::~Butler ()
{
- _stop_thread = true;
+ {
+ boost::mutex::scoped_lock lm (_mutex);
+ _stop_thread = true;
+ }
+
_thread->interrupt ();
try {
_thread->join ();
delete _thread;
}
+/** Caller must hold a lock on _mutex */
bool
Butler::should_run () const
{
- return (_video.size() < VIDEO_READAHEAD || _audio.size() < AUDIO_READAHEAD) && !_stop_thread && !_finished;
+ return (_video.size() < VIDEO_READAHEAD || (!_disable_audio && _audio.size() < AUDIO_READAHEAD)) && !_stop_thread && !_finished && !_died;
}
void
/* Do any seek that has been requested */
if (_pending_seek_position) {
+ _finished = false;
_player->seek (*_pending_seek_position, _pending_seek_accurate);
_pending_seek_position = optional<DCPTime> ();
}
*/
while (should_run() && !_pending_seek_position) {
lm.unlock ();
- if (_player->pass ()) {
+ bool const r = _player->pass ();
+ lm.lock ();
+ if (r) {
_finished = true;
_arrived.notify_all ();
break;
}
- lm.lock ();
_arrived.notify_all ();
}
}
} catch (boost::thread_interrupted) {
/* The butler thread is being terminated */
+ boost::mutex::scoped_lock lm (_mutex);
+ _finished = true;
+ _arrived.notify_all ();
} catch (...) {
store_current ();
- _finished = true;
+ boost::mutex::scoped_lock lm (_mutex);
+ _died = true;
_arrived.notify_all ();
}
boost::mutex::scoped_lock lm (_mutex);
/* Wait for data if we have none */
- while (_video.empty() && !_finished) {
+ while (_video.empty() && !_finished && !_died) {
_arrived.wait (lm);
}
- if (_video.empty() && _finished) {
+ if (_video.empty()) {
return make_pair (shared_ptr<PlayerVideo>(), DCPTime());
}
Butler::seek (DCPTime position, bool accurate)
{
boost::mutex::scoped_lock lm (_mutex);
+ if (_died) {
+ return;
+ }
+
_video.clear ();
_audio.clear ();
_finished = false;
}
void
-Butler::audio (shared_ptr<AudioBuffers> audio, DCPTime time)
+Butler::audio (shared_ptr<AudioBuffers> audio)
{
{
boost::mutex::scoped_lock lm (_mutex);
}
}
- _audio.put (audio, time);
+ _audio.put (remap (audio, _audio_channels, _audio_mapping));
}
void