From 4e60343e552e8b7339678c6df7255a0f91dc769e Mon Sep 17 00:00:00 2001
From: Max Kellermann <max@duempel.org>
Date: Sat, 12 Apr 2008 04:12:53 +0000
Subject: added outputBufferRelative()

The cross-fade check is still very complicated whenever it uses
OutputBuffer internals.  Greatly simplify another check by introducing
outputBufferRelative().

git-svn-id: https://svn.musicpd.org/mpd/trunk@7264 09075e82-0dd4-0310-85a5-a0d7c8717e4f
---
 src/decode.c       |  8 ++------
 src/outputBuffer.c | 13 +++++++++----
 src/outputBuffer.h |  6 ++++++
 3 files changed, 17 insertions(+), 10 deletions(-)

diff --git a/src/decode.c b/src/decode.c
index 688d78322..55f272303 100644
--- a/src/decode.c
+++ b/src/decode.c
@@ -496,12 +496,8 @@ static void decodeParent(PlayerControl * pc, DecoderControl * dc, OutputBuffer *
 		else if (!outputBufferEmpty(cb) && cb->begin != next) {
 			unsigned int fadePosition;
 			if (doCrossFade == 1 && next >= 0 &&
-			    ((next > cb->begin &&
-			      (fadePosition = next - cb->begin)
-			      <= crossFadeChunks) ||
-			     (cb->begin > next &&
-			      (fadePosition = next - cb->begin +
-			       buffered_chunks) <= crossFadeChunks))) {
+			    (fadePosition = outputBufferRelative(cb, next))
+			    <= crossFadeChunks) {
 				/* perform cross fade */
 				if (nextChunk < 0) {
 					/* beginning of the cross fade
diff --git a/src/outputBuffer.c b/src/outputBuffer.c
index f6fb38937..aadd5e379 100644
--- a/src/outputBuffer.c
+++ b/src/outputBuffer.c
@@ -62,12 +62,17 @@ int outputBufferEmpty(const OutputBuffer * cb)
 	return cb->begin == cb->end;
 }
 
-unsigned availableOutputBuffer(const OutputBuffer * cb)
+unsigned int outputBufferRelative(const OutputBuffer * cb, unsigned i)
 {
-	if (cb->end >= cb->begin)
-		return cb->end - cb->begin;
+	if (i >= cb->begin)
+		return i - cb->begin;
 	else
-		return cb->end + buffered_chunks - cb->begin;
+		return i + buffered_chunks - cb->begin;
+}
+
+unsigned availableOutputBuffer(const OutputBuffer * cb)
+{
+	return outputBufferRelative(cb, cb->end);
 }
 
 int outputBufferAbsolute(const OutputBuffer * cb, unsigned relative)
diff --git a/src/outputBuffer.h b/src/outputBuffer.h
index 252a0a905..0af264c14 100644
--- a/src/outputBuffer.h
+++ b/src/outputBuffer.h
@@ -58,6 +58,12 @@ void flushOutputBuffer(OutputBuffer * cb);
 /** is the buffer empty? */
 int outputBufferEmpty(const OutputBuffer * cb);
 
+/**
+ * what is the position of the specified chunk number, relative to
+ * the first chunk in use?
+ */
+unsigned int outputBufferRelative(const OutputBuffer * cb, unsigned i);
+
 /** determine the number of decoded chunks */
 unsigned availableOutputBuffer(const OutputBuffer * cb);
 
-- 
cgit v1.2.3