From 2cc59816a6e6304e643bc5c7ca055dd2567cba9e Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Tue, 1 Jan 2008 10:09:56 +0000 Subject: Simplify decode cleanup logic a bit DECODE_STATE_STOP is always set as dc->state, and dc->stop is always cleared. So handle it in decodeStart once rather than doing it in every plugin. While we're at it, fix a long-standing (but difficult to trigger) bug in mpc_decode where we failed to return if mpc_decoder_initialize() fails. git-svn-id: https://svn.musicpd.org/mpd/trunk@7122 09075e82-0dd4-0310-85a5-a0d7c8717e4f --- src/inputPlugins/mpc_plugin.c | 17 ----------------- 1 file changed, 17 deletions(-) (limited to 'src/inputPlugins/mpc_plugin.c') diff --git a/src/inputPlugins/mpc_plugin.c b/src/inputPlugins/mpc_plugin.c index 46de5b8ea..60fd6fde6 100644 --- a/src/inputPlugins/mpc_plugin.c +++ b/src/inputPlugins/mpc_plugin.c @@ -156,13 +156,9 @@ static int mpc_decode(OutputBuffer * cb, DecoderControl * dc, mpc_streaminfo_init(&info); if ((ret = mpc_streaminfo_read(&info, &reader)) != ERROR_CODE_OK) { - closeInputStream(inStream); if (!dc->stop) { ERROR("Not a valid musepack stream\n"); return -1; - } else { - dc->state = DECODE_STATE_STOP; - dc->stop = 0; } return 0; } @@ -170,13 +166,9 @@ static int mpc_decode(OutputBuffer * cb, DecoderControl * dc, mpc_decoder_setup(&decoder, &reader); if (!mpc_decoder_initialize(&decoder, &info)) { - closeInputStream(inStream); if (!dc->stop) { ERROR("Not a valid musepack stream\n"); return -1; - } else { - dc->state = DECODE_STATE_STOP; - dc->stop = 0; } return 0; } @@ -264,19 +256,10 @@ static int mpc_decode(OutputBuffer * cb, DecoderControl * dc, replayGainInfo); } - closeInputStream(inStream); - flushOutputBuffer(cb); freeReplayGainInfo(replayGainInfo); - if (dc->stop) { - dc->state = DECODE_STATE_STOP; - dc->stop = 0; - } else { - dc->state = DECODE_STATE_STOP; - } - return 0; } -- cgit v1.2.3