Merge pull request #223 from aware70/fix-audio-codec-setup

Fix warning about missing default codecs on first run
This commit is contained in:
Dan Paulat 2024-06-08 15:55:31 -05:00 committed by GitHub
commit 9dfbac6617
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -164,10 +164,9 @@ bool AudioCodecPage::IsRequired()
auto oggCodecs =
oggFormat.supportedAudioCodecs(QMediaFormat::ConversionMode::Decode);
// Setup is required if codec errors are not ignored, and the default codecs
// are not supported
return (!ignoreCodecErrors &&
oggCodecs.contains(QMediaFormat::AudioCodec::Vorbis));
// Setup is required if codec errors are not ignored, and no Ogg support
// is found.
return (!ignoreCodecErrors && oggCodecs.empty());
}
} // namespace setup