From 9b99a9897a96cbc7e7c3697d5c3c4e70ffcf0047 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Sat, 3 Aug 2019 08:09:45 +0200 Subject: [PATCH] decoder/mad: don't count the Xing/LAME metadata frame The Xing/LAME frame indicates how many frames there are, but that excludes the initial Xing/LAME frame. Therefore, it should not be counted. This fixes an off-by-one bug which caused the last frame to be skipped, fixing one part of https://github.com/MusicPlayerDaemon/MPD/issues/601 --- src/decoder/plugins/MadDecoderPlugin.cxx | 1 - 1 file changed, 1 deletion(-) diff --git a/src/decoder/plugins/MadDecoderPlugin.cxx b/src/decoder/plugins/MadDecoderPlugin.cxx index 7b3675e37..33a9be644 100644 --- a/src/decoder/plugins/MadDecoderPlugin.cxx +++ b/src/decoder/plugins/MadDecoderPlugin.cxx @@ -905,7 +905,6 @@ MadDecoder::Read() noexcept case MadDecoderMuteFrame::SKIP: mute_frame = MadDecoderMuteFrame::NONE; - UpdateTimerNextFrame(); break; case MadDecoderMuteFrame::SEEK: if (elapsed_time >= seek_time)