Minor warnings/lints

Author: WorkingRobot

Craftimizer/ImGuiUtils.cs

@@ -356,7 +356,7 @@ private sealed class SearchableComboData<T> where T : IEquatable<T>
 
         public SearchableComboData(IEnumerable<T> items, T selectedItem, Func<T, string> getString)
         {
-            this.items = items.ToImmutableArray();
+            this.items = [.. items];
             filteredItems = [selectedItem];
             this.selectedItem = selectedItem;
             this.getString = getString;

Craftimizer/Windows/MacroClipboard.cs

@@ -19,7 +19,7 @@ public sealed class MacroClipboard : Window, IDisposable
 
     public MacroClipboard(IEnumerable<string> macros) : base("Macro Clipboard", WindowFlags)
     {
-        Macros = new(macros);
+        Macros = [.. macros];
 
         IsOpen = true;
         AllowPinning = false;

Craftimizer/Windows/MacroEditor.cs

@@ -719,9 +719,19 @@ private readonly struct RecipeWrapper(Recipe recipe) : IEquatable<RecipeWrapper>
 
         public bool Equals(RecipeWrapper other) =>
             Recipe.RowId == other.Recipe.RowId;
+
+        public override bool Equals(object? obj)
+        {
+            return obj is RecipeWrapper other && Equals(other);
+        }
+
+        public override int GetHashCode()
+        {
+            return unchecked((int)Recipe.RowId);
+        }
     }
 
-    private readonly List<RecipeWrapper> searchableRecipes = LuminaSheets.RecipeSheet.Where(r => r.RecipeLevelTable.RowId != 0 && r.ItemResult.RowId != 0).Select(r => new RecipeWrapper(r)).ToList();
+    private readonly List<RecipeWrapper> searchableRecipes = [.. LuminaSheets.RecipeSheet.Where(r => r.RecipeLevelTable.RowId != 0 && r.ItemResult.RowId != 0).Select(r => new RecipeWrapper(r))];
 
     private bool DrawRecipeParams()
     {

Craftimizer/Windows/MacroList.cs

@@ -343,7 +343,7 @@ private void RefreshSearch()
     {
         if (string.IsNullOrWhiteSpace(searchText))
         {
-            sortedMacros = new(Macros);
+            sortedMacros = [.. Macros];
             isUnsorted = true;
             return;
         }

Craftimizer/Windows/RecipeNote.cs

@@ -1191,7 +1191,7 @@ private void CalculateSavedMacro()
                 config = config.FilterSpecialistActions();
             var mctsConfig = new MCTSConfig(config);
             var simulator = new SimulatorNoRandom();
-            List<Macro> macros = new(Service.Configuration.Macros);
+            List<Macro> macros = [.. Service.Configuration.Macros];
 
             token.ThrowIfCancellationRequested();
 

Craftimizer/Windows/Settings.cs

@@ -77,6 +77,7 @@ private static void DrawOption<T>(string label, string tooltip, T value, T min,
     {
         ImGui.SetNextItemWidth(OptionWidth);
         var text = value.ToString();
+        ArgumentNullException.ThrowIfNull(text, nameof(value));
         if (ImGui.InputText(label, ref text, 8, ImGuiInputTextFlags.AutoSelectAll | ImGuiInputTextFlags.CharsDecimal))
         {
             if (T.TryParse(text, null, out var newValue))
@@ -791,7 +792,7 @@ private static void DrawActionPool(ref ActionType[] actionPool, float poolWidth,
         isDirty = false;
 
         var recipeData = Service.Plugin.GetDefaultStats().Recipe;
-        HashSet<ActionType> pool = new(actionPool);
+        HashSet<ActionType> pool = [.. actionPool];
 
         var imageSize = ImGui.GetFrameHeight() * 2;
         var spacing = ImGui.GetStyle().ItemSpacing.Y;