aac: moved code to aac_buffer_shift()

Shifting from the buffer queue is a common operation, and should be provided as a separate function. Move code to aac_buffer_shift() and add a bunch of assertions.

aac: moved code to aac_buffer_shift()
b7ad3e41 · Max Kellermann · a3cc928c · b7ad3e41
Commit b7ad3e41 authored Aug 26, 2008 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 14 additions and 7 deletions

aac_plugin.c src/inputPlugins/aac_plugin.c +14 -7

No files found.
--- a/src/inputPlugins/aac_plugin.c
+++ b/src/inputPlugins/aac_plugin.c
@@ -37,6 +37,19 @@ typedef struct {
 	int atEof;
 } AacBuffer;

+static void aac_buffer_shift(AacBuffer * b, size_t length)
+{
+	assert(length >= b->bytesConsumed);
+	assert(length <= b->bytesConsumed + b->bytesIntoBuffer);
+
+	memmove(b->buffer, b->buffer + length,
+		b->bytesConsumed + b->bytesIntoBuffer - length);
+
+	length -= b->bytesConsumed;
+	b->bytesConsumed = 0;
+	b->bytesIntoBuffer -= length;
+}
+
 static void fillAacBuffer(AacBuffer * b)
 {
 	size_t bread;
@@ -45,13 +58,7 @@ static void fillAacBuffer(AacBuffer * b)
 		/* buffer already full */
 		return;

-	if (b->bytesConsumed > 0 && b->bytesIntoBuffer > 0) {
-		memmove((void *)b->buffer, (void *)(b->buffer +
-						    b->bytesConsumed),
-			b->bytesIntoBuffer);
-	}
-
-	b->bytesConsumed = 0;
+	aac_buffer_shift(b, b->bytesConsumed);

 	if (!b->atEof) {
 		size_t rest = FAAD_MIN_STREAMSIZE * AAC_MAX_CHANNELS -