diff options
| author | gnattu <gnattuoc@me.com> | 2024-07-18 01:59:16 +0800 |
|---|---|---|
| committer | gnattu <gnattuoc@me.com> | 2024-07-29 06:23:16 +0800 |
| commit | 79c4469ac7ac936ded42e0d855df10a0a7d609e2 (patch) | |
| tree | e0fbc3877f738afcbe570c61390a7078eb303c55 /MediaBrowser.Providers/MediaInfo | |
| parent | 5c5b326b1aa794e3956cc81c08dd380ac9fd9ae0 (diff) | |
Remove redundant NaN check
Signed-off-by: gnattu <gnattuoc@me.com>
Diffstat (limited to 'MediaBrowser.Providers/MediaInfo')
| -rw-r--r-- | MediaBrowser.Providers/MediaInfo/AudioFileProber.cs | 20 |
1 files changed, 5 insertions, 15 deletions
diff --git a/MediaBrowser.Providers/MediaInfo/AudioFileProber.cs b/MediaBrowser.Providers/MediaInfo/AudioFileProber.cs index 5270b1c3e..7e0773b6d 100644 --- a/MediaBrowser.Providers/MediaInfo/AudioFileProber.cs +++ b/MediaBrowser.Providers/MediaInfo/AudioFileProber.cs @@ -291,7 +291,6 @@ namespace MediaBrowser.Providers.MediaInfo } track.AdditionalFields.TryGetValue("REPLAYGAIN_TRACK_GAIN", out var trackGainTag); - float trackGain = float.NaN; if (trackGainTag is not null) { @@ -302,17 +301,11 @@ namespace MediaBrowser.Providers.MediaInfo if (float.TryParse(trackGainTag, NumberStyles.Float, CultureInfo.InvariantCulture, out var value)) { - trackGain = value; + audio.NormalizationGain = value; } } - if (!float.IsNaN(trackGain)) - { - audio.NormalizationGain = trackGain; - } - - if ((options.ReplaceAllMetadata || !audio.TryGetProviderId(MetadataProvider.MusicBrainzArtist, out _)) - && !string.IsNullOrEmpty(tags.MusicBrainzArtistId)) + if (options.ReplaceAllMetadata || !audio.TryGetProviderId(MetadataProvider.MusicBrainzArtist, out _)) { if ((track.AdditionalFields.TryGetValue("MUSICBRAINZ_ARTISTID", out var musicBrainzArtistTag) || track.AdditionalFields.TryGetValue("MusicBrainz Artist Id", out musicBrainzArtistTag)) @@ -322,8 +315,7 @@ namespace MediaBrowser.Providers.MediaInfo } } - if ((options.ReplaceAllMetadata || !audio.TryGetProviderId(MetadataProvider.MusicBrainzAlbumArtist, out _)) - && !string.IsNullOrEmpty(tags.MusicBrainzReleaseArtistId)) + if (options.ReplaceAllMetadata || !audio.TryGetProviderId(MetadataProvider.MusicBrainzAlbumArtist, out _)) { if ((track.AdditionalFields.TryGetValue("MUSICBRAINZ_ALBUMARTISTID", out var musicBrainzReleaseArtistIdTag) || track.AdditionalFields.TryGetValue("MusicBrainz Album Artist Id", out musicBrainzReleaseArtistIdTag)) @@ -333,8 +325,7 @@ namespace MediaBrowser.Providers.MediaInfo } } - if ((options.ReplaceAllMetadata || !audio.TryGetProviderId(MetadataProvider.MusicBrainzAlbum, out _)) - && !string.IsNullOrEmpty(tags.MusicBrainzReleaseId)) + if (options.ReplaceAllMetadata || !audio.TryGetProviderId(MetadataProvider.MusicBrainzAlbum, out _)) { if ((track.AdditionalFields.TryGetValue("MUSICBRAINZ_ALBUMID", out var musicBrainzReleaseIdTag) || track.AdditionalFields.TryGetValue("MusicBrainz Album Id", out musicBrainzReleaseIdTag)) @@ -344,8 +335,7 @@ namespace MediaBrowser.Providers.MediaInfo } } - if ((options.ReplaceAllMetadata || !audio.TryGetProviderId(MetadataProvider.MusicBrainzReleaseGroup, out _)) - && !string.IsNullOrEmpty(tags.MusicBrainzReleaseGroupId)) + if (options.ReplaceAllMetadata || !audio.TryGetProviderId(MetadataProvider.MusicBrainzReleaseGroup, out _)) { if ((track.AdditionalFields.TryGetValue("MUSICBRAINZ_RELEASEGROUPID", out var musicBrainzReleaseGroupIdTag) || track.AdditionalFields.TryGetValue("MusicBrainz Release Group Id", out musicBrainzReleaseGroupIdTag)) |