From a07ab27dae485c253d5f00677ea6ca94c26d7741 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Sun, 11 Oct 2009 23:44:21 +0200 Subject: [PATCH] decoder_thread: removed redundant NULL assignments The while() clause resets the "plugin" variable. We don't need to reset it at the end of the loop body. --- src/decoder_thread.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/decoder_thread.c b/src/decoder_thread.c index 9124db033..5140f2ec3 100644 --- a/src/decoder_thread.c +++ b/src/decoder_thread.c @@ -164,7 +164,7 @@ static void decoder_run_song(const struct song *song, const char *uri) if (ret) break; - plugin = NULL; + assert(dc.state == DECODE_STATE_START); } /* if that fails, try suffix matching the URL: */ @@ -180,7 +180,6 @@ static void decoder_run_song(const struct song *song, const char *uri) break; assert(dc.state == DECODE_STATE_START); - plugin = NULL; } } /* fallback to mp3: */