added output_buffer_skip()

First patch without camelCase ;)

output_buffer_skip() lets us eliminate advanceOutputBufferTo(), and
removes yet another external OutputBuffer struct access.

git-svn-id: https://svn.musicpd.org/mpd/trunk@7312 09075e82-0dd4-0310-85a5-a0d7c8717e4f

src/decode.c

@@ -361,11 +361,6 @@ void decoderInit(void)
 		FATAL("Failed to spawn decoder task: %s\n", strerror(errno));
 }
 
-static void advanceOutputBufferTo(OutputBuffer * cb, int to)
-{
-	cb->begin = to;
-}
-
 static void crossFade(OutputBufferChunk * a, OutputBufferChunk * b,
 		      AudioFormat * format,
 		      unsigned int fadePosition, unsigned int crossFadeChunks)
@@ -576,9 +571,7 @@ static void decodeParent(PlayerControl * pc, DecoderControl * dc, OutputBuffer *
 				/* the cross-fade is finished; skip
 				   the section which was cross-faded
 				   (and thus already played) */
-				nextChunk = outputBufferAbsolute(cb, crossFadeChunks);
+				output_buffer_skip(cb, crossFadeChunks);
-				if (nextChunk >= 0)
-					advanceOutputBufferTo(cb, nextChunk);
 			}
 
 			doCrossFade = 0;

src/outputBuffer.c

@@ -218,3 +218,9 @@ int sendDataToOutputBuffer(OutputBuffer * cb, InputStream * inStream,
 	return 0;
 }
 
+void output_buffer_skip(OutputBuffer * cb, unsigned num)
+{
+	int i = outputBufferAbsolute(cb, num);
+	if (i >= 0)
+		cb->begin = i;
+}

src/outputBuffer.h

@@ -99,4 +99,6 @@ int sendDataToOutputBuffer(OutputBuffer * cb,
 			   float data_time,
 			   mpd_uint16 bitRate, ReplayGainInfo * replayGainInfo);
 
+void output_buffer_skip(OutputBuffer * cb, unsigned num);
+
 #endif