h1or4dme8/osu-lazer
1
0
Fork 0
mirror of https://github.com/ppy/osu.git synced 2025-01-28 02:02:53 +08:00
Code Issues Releases Wiki Activity

Merge pull request #18936 from peppy/fix-nan-diff-calc-result

Browse Source 
Guard against `NaN` star difficulty results
This commit is contained in:
Dan Balasescu 2022-06-29 21:51:29 +09:00 committed by GitHub
commit 68d4676ad0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
osu.Game/Beatmaps/BeatmapDifficultyCache.cs
View File

@ -207,7 +207,7 @@ namespace osu.Game.Beatmaps
if (cancellationToken.IsCancellationRequested)
return;
var starDifficulty = task.GetResultSafely();
StarDifficulty? starDifficulty = task.GetResultSafely();
if (starDifficulty != null)
bindable.Value = starDifficulty.Value;

4
osu.Game/Beatmaps/StarDifficulty.cs
View File

@ -34,7 +34,7 @@ namespace osu.Game.Beatmaps
/// </summary>
public StarDifficulty([NotNull] DifficultyAttributes attributes)
{
Stars = attributes.StarRating;
Stars = double.IsFinite(attributes.StarRating) ? attributes.StarRating : 0;
MaxCombo = attributes.MaxCombo;
Attributes = attributes;
// Todo: Add more members (BeatmapInfo.DifficultyRating? Attributes? Etc...)
@ -46,7 +46,7 @@ namespace osu.Game.Beatmaps
/// </summary>
public StarDifficulty(double starDifficulty, int maxCombo)
{
Stars = starDifficulty;
Stars = double.IsFinite(starDifficulty) ? starDifficulty : 0;
MaxCombo = maxCombo;
Attributes = null;
}