sim1222/osukey
1
1
Fork 0
mirror of https://github.com/osukey/osukey.git synced 2025-07-02 00:40:09 +09:00
Code Issues Packages Projects Releases Wiki Activity

Use CreateCopy instead of Clone interface

Browse Source 
I was going for conformity by using the IClonable interface, but it
doesn't look like we use it anywhere else in the project.
This commit is contained in:
Dean Herbert
2021-01-05 13:13:51 +09:00
parent b7dd54847f
commit caa88c6100
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
osu.Game/Beatmaps/ControlPoints/ControlPointInfo.cs
View File

@ -11,7 +11,7 @@ using osu.Framework.Lists;
namespace osu.Game.Beatmaps.ControlPoints
{
[Serializable]
public class ControlPointInfo : ICloneable
public class ControlPointInfo
{
/// <summary>
/// All control points grouped by time.
@ -298,7 +298,7 @@ namespace osu.Game.Beatmaps.ControlPoints
}
}
public object Clone()
public ControlPointInfo CreateCopy()
{
var controlPointInfo = new ControlPointInfo();

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

@ -112,7 +112,7 @@ namespace osu.Game.Beatmaps
// Convert
IBeatmap converted = converter.Convert(cancellationSource.Token);
converted.ControlPointInfo = (ControlPointInfo)converted.ControlPointInfo.Clone();
converted.ControlPointInfo = converted.ControlPointInfo.CreateCopy();
// Apply conversion mods to the result
foreach (var mod in mods.OfType<IApplicableAfterBeatmapConversion>())