decoder/faad: eliminate the adts_find_frame() loop

This loop is completely unnecessary. We just need to find the first ADTS frame and feed it into NeAACDecInit().

decoder/faad: eliminate the adts_find_frame() loop
e42b1520 · Max Kellermann · da599e3f · e42b1520
Commit e42b1520 authored Jul 12, 2014 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 5 deletions

FaadDecoderPlugin.cxx src/decoder/plugins/FaadDecoderPlugin.cxx +2 -5

No files found.
--- a/src/decoder/plugins/FaadDecoderPlugin.cxx
+++ b/src/decoder/plugins/FaadDecoderPlugin.cxx
@@ -365,11 +365,8 @@ faad_stream_decode(Decoder &mpd_decoder, InputStream &is,
 {
 	const float total_time = faad_song_duration(buffer, is);

-	while (!decoder_buffer_is_full(buffer) && !is.LockIsEOF() &&
-	       decoder_get_command(mpd_decoder) == DecoderCommand::NONE) {
-		adts_find_frame(buffer);
-		decoder_buffer_fill(buffer);
-	}
+	if (adts_find_frame(buffer) == 0)
+		return;

 	/* initialize it */