modplug: removed EOF check from the while loop

EOF is checked by input_stream_read() (decoder_read() here). Don't do it twice. The check was wrong anyway, it was reversed.

modplug: removed EOF check from the while loop
0dc1b4a4 · Max Kellermann · 793934cf · 0dc1b4a4
Commit 0dc1b4a4 authored Jan 24, 2009 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 2 deletions

modplug_plugin.c src/decoder/modplug_plugin.c +4 -2

No files found.
--- a/src/decoder/modplug_plugin.c
+++ b/src/decoder/modplug_plugin.c
@@ -55,7 +55,8 @@ static GByteArray *mod_loadfile(struct decoder *decoder, struct input_stream *is
 	}

 	data = g_malloc(MODPLUG_READ_BLOCK);
-	do {
+
+	while (true) {
 		ret = decoder_read(decoder, is, data, MODPLUG_READ_BLOCK);
 		if (ret == 0) {
 			if (input_stream_eof(is))
@@ -76,7 +77,8 @@ static GByteArray *mod_loadfile(struct decoder *decoder, struct input_stream *is
 		}

 		g_byte_array_append(bdatas, data, ret);
-	} while (input_stream_eof(is));
+	}
+
 	g_free(data);

 	return bdatas;