From 7e838c80420d35a9c633cfc038fd3afd7ecf728a Mon Sep 17 00:00:00 2001
From: Salman Ahmed <email@iisalman.me>
Date: Sat, 22 Aug 2020 13:07:15 +0300
Subject: [PATCH] Add comment explaining why direct string lookups are used

---
 osu.Game/Tests/Visual/SkinnableTestScene.cs | 1 +
 1 file changed, 1 insertion(+)

diff --git a/osu.Game/Tests/Visual/SkinnableTestScene.cs b/osu.Game/Tests/Visual/SkinnableTestScene.cs
index 49ba02fdea..a9e3d36ca0 100644
--- a/osu.Game/Tests/Visual/SkinnableTestScene.cs
+++ b/osu.Game/Tests/Visual/SkinnableTestScene.cs
@@ -160,6 +160,7 @@ namespace osu.Game.Tests.Visual
             {
                 this.extrapolateAnimations = extrapolateAnimations;
 
+                // Use a direct string lookup for simplicity, as they're legacy settings and not worth creating enums for them.
                 legacyFontPrefixes.Add(GetConfig<string, string>("HitCirclePrefix")?.Value ?? "default");
                 legacyFontPrefixes.Add(GetConfig<string, string>("ScorePrefix")?.Value ?? "score");
                 legacyFontPrefixes.Add(GetConfig<string, string>("ComboPrefix")?.Value ?? "score");