h1or4dme8/osu-lazer
1
0
Fork 0
mirror of https://github.com/ppy/osu.git synced 2024-12-13 03:35:35 +08:00
Code Issues Releases Wiki Activity

Remove unnecessary AllowThousands flag

Browse Source 
The flag is there to match `float.Parse` behaviour, but it's too illogical and unnecessary to have it.
This commit is contained in:
Salman Ahmed 2024-01-10 17:48:47 +03:00
parent 698ae66a49
commit 7ca4d85441
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
osu.Game/Skinning/LegacyManiaSkinDecoder.cs
View File

@ -155,7 +155,7 @@ namespace osu.Game.Skinning
if (i >= output.Length)
break;
if (!float.TryParse(values[i], NumberStyles.Float | NumberStyles.AllowThousands, CultureInfo.InvariantCulture, out float parsedValue))
if (!float.TryParse(values[i], NumberStyles.Float, CultureInfo.InvariantCulture, out float parsedValue))
// some skins may provide incorrect entries in array values. to match stable behaviour, read such entries as zero.
// see: https://github.com/ppy/osu/issues/26464, stable code: https://github.com/peppy/osu-stable-reference/blob/3ea48705eb67172c430371dcfc8a16a002ed0d3d/osu!/Graphics/Skinning/Components/Section.cs#L134-L137
parsedValue = 0;