From 977e1a3bfec706663347ecb167219971ee738e9f Mon Sep 17 00:00:00 2001
From: mcendu <nathandu@outlook.com>
Date: Tue, 31 Mar 2020 17:48:37 +0800
Subject: [PATCH] split shortcut into two methods

---
 .../Skinning/LegacyManiaColumnElement.cs        | 17 +++++++++++++----
 1 file changed, 13 insertions(+), 4 deletions(-)

diff --git a/osu.Game.Rulesets.Mania/Skinning/LegacyManiaColumnElement.cs b/osu.Game.Rulesets.Mania/Skinning/LegacyManiaColumnElement.cs
index bf7405bb44..5386d05504 100644
--- a/osu.Game.Rulesets.Mania/Skinning/LegacyManiaColumnElement.cs
+++ b/osu.Game.Rulesets.Mania/Skinning/LegacyManiaColumnElement.cs
@@ -40,14 +40,23 @@ namespace osu.Game.Rulesets.Mania.Skinning
             }
         }
 
+        /// <summary>
+        /// Retrieve a per-column-count skin configuration.
+        /// </summary>
+        /// <param name="skin">The skin from which configuration is retrieved.</param>
+        /// <param name="lookup">The value to retrieve.</param>
+        /// <param name="index">If not null, denotes the index of the column to which the entry applies.</param>
+        protected IBindable<T> GetManiaSkinConfig<T>(ISkin skin, LegacyManiaSkinConfigurationLookups lookup, int? index = null)
+            => skin.GetConfig<LegacyManiaSkinConfigurationLookup, T>(
+                new LegacyManiaSkinConfigurationLookup(Stage?.Columns.Count ?? 4, lookup, index));
+
         /// <summary>
         /// Retrieve a per-column skin configuration.
         /// </summary>
         /// <param name="skin">The skin from which configuration is retrieved.</param>
         /// <param name="lookup">The value to retrieve.</param>
-        /// <param name="index">The index of the column to which the entry applies.</param>
-        protected IBindable<T> GetManiaSkinConfig<T>(ISkin skin, LegacyManiaSkinConfigurationLookups lookup, int? index = null)
-            => skin.GetConfig<LegacyManiaSkinConfigurationLookup, T>(
-                new LegacyManiaSkinConfigurationLookup(Stage?.Columns.Count ?? 4, lookup, index ?? Column.Index));
+        /// <param name="index">The index of the column to which the entry applies. Defaults to the column index.</param>
+        protected IBindable<T> GetPerColumnSkinConfig<T>(ISkin skin, LegacyManiaSkinConfigurationLookups lookup, int? index = null)
+            => GetManiaSkinConfig<T>(skin, lookup, index ?? Column.Index);
     }
 }