Setting: change max recently loaded sav file count

kwsch · Mar 17, 2024 · 3dd6089 · 3dd6089
1 parent c3c7fac
commit 3dd6089
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 7 deletions.
diff --git a/PKHeX.WinForms/Properties/PKHeXSettings.cs b/PKHeX.WinForms/Properties/PKHeXSettings.cs
@@ -141,7 +141,18 @@ public sealed class StartupSettings : IStartupSettings
     public PluginLoadSetting PluginLoadMethod { get; set; } = PluginLoadSetting.LoadFrom;
 
     [Browsable(false)]
-    public List<string> RecentlyLoaded { get; set; } = new(MaxRecentCount);
+    public List<string> RecentlyLoaded { get; set; } = new(DefaultMaxRecent);
+
+    private const int DefaultMaxRecent = 10;
+    private uint MaxRecentCount = DefaultMaxRecent;
+
+    [LocalizedDescription("Amount of recently loaded save files to remember.")]
+    public uint RecentlyLoadedMaxCount
+    {
+        get => MaxRecentCount;
+        // Sanity check to not let the user foot-gun themselves a slow recall time.
+        set => MaxRecentCount = Math.Clamp(value, 1, 1000);
+    }
 
     // Don't let invalid values slip into the startup version.
     private GameVersion _defaultSaveVersion = PKX.Version;
@@ -171,8 +182,6 @@ public GameVersion DefaultSaveVersion
         }
     }
 
-    private const int MaxRecentCount = 10;
-
     public void LoadSaveFile(string path)
     {
         var recent = RecentlyLoaded;

diff --git a/PKHeX.WinForms/Subforms/SAV_FolderList.cs b/PKHeX.WinForms/Subforms/SAV_FolderList.cs
@@ -130,8 +130,6 @@ private static IEnumerable<CustomFolderPath> GetConsolePaths(IEnumerable<string>
             return [];
 
         var root = Path.GetPathRoot(path3DS);
-        // ReSharper disable once ConditionIsAlwaysTrueOrFalse
-        // ReSharper disable once HeuristicUnreachableCode
         if (root == null)
             return [];
 
@@ -146,8 +144,6 @@ private static IEnumerable<CustomFolderPath> GetSwitchPaths(IEnumerable<string>
             return [];
 
         var root = Path.GetPathRoot(pathNX);
-        // ReSharper disable once ConditionIsAlwaysTrueOrFalse
-        // ReSharper disable once HeuristicUnreachableCode
         if (root == null)
             return [];