From 06c60888dc982ad66526fbe14aa21318e7d62958 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Tue, 11 Nov 2008 21:39:50 +0100 Subject: mpc: eliminated the local variable "eof" Since each "eof=true" is followed by "break", the variable is superfluous. --- src/decoder/mpc_plugin.c | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/src/decoder/mpc_plugin.c b/src/decoder/mpc_plugin.c index 584b9fed6..2496f51a9 100644 --- a/src/decoder/mpc_plugin.c +++ b/src/decoder/mpc_plugin.c @@ -108,7 +108,6 @@ mpc_decode(struct decoder *mpd_decoder, struct input_stream *inStream) MPC_SAMPLE_FORMAT sample_buffer[MPC_DECODER_BUFFER_LENGTH]; - bool eof = false; long ret; #define MPC_CHUNK_SIZE 4096 char chunk[MPC_CHUNK_SIZE]; @@ -162,7 +161,7 @@ mpc_decode(struct decoder *mpd_decoder, struct input_stream *inStream) inStream->seekable, mpc_streaminfo_get_length(&info)); - while (!eof) { + while (true) { if (decoder_get_command(mpd_decoder) == DECODE_COMMAND_SEEK) { samplePos = decoder_seek_where(mpd_decoder) * audio_format.sample_rate; @@ -179,10 +178,8 @@ mpc_decode(struct decoder *mpd_decoder, struct input_stream *inStream) ret = mpc_decoder_decode(&decoder, sample_buffer, &vbrUpdateAcc, &vbrUpdateBits); - if (ret <= 0 || decoder_get_command(mpd_decoder) == DECODE_COMMAND_STOP) { - eof = true; + if (ret <= 0 || decoder_get_command(mpd_decoder) == DECODE_COMMAND_STOP) break; - } samplePos += ret; @@ -207,10 +204,8 @@ mpc_decode(struct decoder *mpd_decoder, struct input_stream *inStream) chunkpos = 0; dest = (int32_t *)chunk; - if (decoder_get_command(mpd_decoder) == DECODE_COMMAND_STOP) { - eof = true; + if (decoder_get_command(mpd_decoder) == DECODE_COMMAND_STOP) break; - } } } } -- cgit v1.2.3