Rename Film::player to reflect the fact that it creates a new Player on each call.

4 files changed, 4 insertions, 4 deletions

diff --git a/src/lib/film.cc b/src/lib/film.cc
index d216e303f..eb86775e0 100644
--- a/src/lib/film.cc
+++ b/src/lib/film.cc
@@ -706,7 +706,7 @@ Film::have_dcp () const
 }
 
 shared_ptr<Player>
-Film::player () const
+Film::make_player () const
 {
 	return shared_ptr<Player> (new Player (shared_from_this (), _playlist));
 }
diff --git a/src/lib/film.h b/src/lib/film.h
index bf89c6f2c..24c43419a 100644
--- a/src/lib/film.h
+++ b/src/lib/film.h
@@ -98,7 +98,7 @@ public:
 
 	bool have_dcp () const;
 
-	boost::shared_ptr<Player> player () const;
+	boost::shared_ptr<Player> make_player () const;
 	boost::shared_ptr<Playlist> playlist () const;
 
 	OutputAudioFrame dcp_audio_frame_rate () const;
diff --git a/src/lib/transcoder.cc b/src/lib/transcoder.cc
index f4a52639a..7022965bd 100644
--- a/src/lib/transcoder.cc
+++ b/src/lib/transcoder.cc
@@ -64,7 +64,7 @@ audio_proxy (weak_ptr<Encoder> encoder, shared_ptr<const AudioBuffers> audio)
  */
 Transcoder::Transcoder (shared_ptr<const Film> f, shared_ptr<Job> j)
 	: _job (j)
-	, _player (f->player ())
+	, _player (f->make_player ())
 	, _encoder (new Encoder (f, j))
 {
 	_player->Video.connect (bind (video_proxy, _encoder, _1, _2));
diff --git a/src/wx/film_viewer.cc b/src/wx/film_viewer.cc
index cd331a25a..d42829880 100644
--- a/src/wx/film_viewer.cc
+++ b/src/wx/film_viewer.cc
@@ -126,7 +126,7 @@ FilmViewer::set_film (shared_ptr<Film> f)
 		return;
 	}
 
-	_player = f->player ();
+	_player = f->make_player ();
 	_player->disable_audio ();
 	_player->Video.connect (boost::bind (&FilmViewer::process_video, this, _1, _3));
 	_player->Changed.connect (boost::bind (&FilmViewer::player_changed, this, _1));