From d186260016581a76205be6b34279a6c2a4a7dee4 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Tue, 26 Aug 2008 08:27:06 +0200 Subject: use break instead of local variable "eof" Similar to previous patch: eliminate one variable by using "break". This also simplifies the code since we can remove one level of indent. --- src/inputPlugins/audiofile_plugin.c | 28 ++++++++++++---------------- 1 file changed, 12 insertions(+), 16 deletions(-) (limited to 'src') diff --git a/src/inputPlugins/audiofile_plugin.c b/src/inputPlugins/audiofile_plugin.c index a50061d2b..6db342f82 100644 --- a/src/inputPlugins/audiofile_plugin.c +++ b/src/inputPlugins/audiofile_plugin.c @@ -88,10 +88,10 @@ static int audiofile_decode(struct decoder * decoder, char *path) decoder_initialized(decoder, &audio_format, total_time); { - int ret, eof = 0, current = 0; + int ret, current = 0; char chunk[CHUNK_SIZE]; - while (!eof) { + do { if (dc.command == DECODE_COMMAND_SEEK) { decoder_clear(decoder); current = dc.seekWhere * @@ -104,20 +104,16 @@ static int audiofile_decode(struct decoder * decoder, char *path) afReadFrames(af_fp, AF_DEFAULT_TRACK, chunk, CHUNK_SIZE / fs); if (ret <= 0) - eof = 1; - else { - current += ret; - decoder_data(decoder, NULL, - 1, - chunk, ret * fs, - (float)current / - (float)audio_format. - sampleRate, bitRate, - NULL); - if (dc.command == DECODE_COMMAND_STOP) - break; - } - } + + current += ret; + decoder_data(decoder, NULL, + 1, + chunk, ret * fs, + (float)current / + (float)audio_format. + sampleRate, bitRate, + NULL); + } while (dc.command != DECODE_COMMAND_STOP); decoder_flush(decoder); } -- cgit v1.2.3