A previous commit took care to make Decoder::position() not be updated

if the data that was emitted from the decoder was not taken by the player.

This means that when the decoder moves into its end trim the position will
stay where it is (since the player does not take the data).

I can't see the point of doing this; the only use of Decoder::position()
is to decide what to pass() next (I think).

It is also inconvenient because we would like to check Decoder::position()
to decide whether to stop passing a decoder since it's in its end trim
(not doing this causes #1154).

1 files changed, 2 insertions, 2 deletions

diff --git a/src/lib/audio_decoder.h b/src/lib/audio_decoder.h
index ba1520ef5..69655cceb 100644
--- a/src/lib/audio_decoder.h
+++ b/src/lib/audio_decoder.h
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2012-2017 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-2018 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -54,7 +54,7 @@ public:
 	ContentTime stream_position (AudioStreamPtr stream) const;
 
 	/** @return Number of frames of data that were accepted */
-	boost::signals2::signal<Frame (AudioStreamPtr, ContentAudio)> Data;
+	boost::signals2::signal<void (AudioStreamPtr, ContentAudio)> Data;
 
 private:
 	void silence (int milliseconds);