Since we have merged dc->stop, dc->seek into one variable, we don't
have to check both conditions at a time; we can replace "!stop &&
!seek" with "none".
---
src/inputPlugins/mp3_plugin.c | 10 +++-------
1 files changed, 3 insertions(+), 7 deletions(-)
diff --git a/src/inputPlugins/mp3_plugin.c b/src/inputPlugins/mp3_plugin.c
index 27d0fab..dfd6ef0 100644
--- a/src/inputPlugins/mp3_plugin.c
+++ b/src/inputPlugins/mp3_plugin.c
@@ -983,19 +983,15 @@ static int mp3Read(mp3DecodeData * data, struct decoder
*decoder,
decodeNextFrameHeader(data, NULL,
replayGainInfo)) == DECODE_CONT
&& dc.command != DECODE_COMMAND_STOP) ;
- if (ret == DECODE_BREAK ||
- dc.command == DECODE_COMMAND_STOP ||
- dc.command == DECODE_COMMAND_SEEK)
+ if (ret == DECODE_BREAK || dc.command != DECODE_COMMAND_NONE)
break;
else if (ret == DECODE_SKIP)
skip = 1;
if (!data->muteFrame) {
while ((ret = decodeNextFrame(data)) == DECODE_CONT &&
- dc.command != DECODE_COMMAND_STOP &&
- dc.command != DECODE_COMMAND_SEEK) ;
+ dc.command == DECODE_COMMAND_NONE) ;
if (ret == DECODE_BREAK ||
- dc.command == DECODE_COMMAND_STOP ||
- dc.command == DECODE_COMMAND_SEEK)
+ dc.command != DECODE_COMMAND_NONE)
break;
}
if (!skip && ret == DECODE_OK)
-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Musicpd-dev-team mailing list
Musicpd-dev-team@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/musicpd-dev-team
