decoder/mad: submit MixRamp only if there is actual data

Fixes MixRamp failures when a MP3 file has two ID3 tags, one of them without MixRamp.

decoder/mad: submit MixRamp only if there is actual data
cb5c6259 · Max Kellermann · bf287fef · cb5c6259
Commit cb5c6259 authored Dec 01, 2021 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

MadDecoderPlugin.cxx src/decoder/plugins/MadDecoderPlugin.cxx +3 -1

No files found.
--- a/src/decoder/plugins/MadDecoderPlugin.cxx
+++ b/src/decoder/plugins/MadDecoderPlugin.cxx
@@ -310,7 +310,9 @@ MadDecoder::ParseId3(size_t tagsize, Tag *mpd_tag) noexcept
 			found_replay_gain = true;
 		}

-		client->SubmitMixRamp(Id3ToMixRampInfo(id3_tag.get()));
+		if (auto mix_ramp = Id3ToMixRampInfo(id3_tag.get());
+		    mix_ramp.IsDefined())
+			client->SubmitMixRamp(std::move(mix_ramp));
 	}

 #else /* !ENABLE_ID3TAG */