From 42238538bd8daa2d0aed00cd7e8f33237a07c06f Mon Sep 17 00:00:00 2001 From: Jesse Plamondon-Willard Date: Thu, 26 Mar 2026 22:42:35 -0400 Subject: [PATCH] remove `Distinct()` in inventory tracking These cover very rare edge cases with invalid inventory data, so it's not really worth the added performance hit of handling them. The next commits will avoid the error if duplicates do occur. --- src/SMAPI/Framework/StateTracking/ChestTracker.cs | 1 - src/SMAPI/Framework/StateTracking/PlayerTracker.cs | 1 - 2 files changed, 2 deletions(-) diff --git a/src/SMAPI/Framework/StateTracking/ChestTracker.cs b/src/SMAPI/Framework/StateTracking/ChestTracker.cs index 5a6c4c56..9b20b6a5 100644 --- a/src/SMAPI/Framework/StateTracking/ChestTracker.cs +++ b/src/SMAPI/Framework/StateTracking/ChestTracker.cs @@ -48,7 +48,6 @@ internal class ChestTracker : IDisposable this.StackSizes = this.Chest.Items .Where(n => n != null) - .Distinct() .ToDictionary(n => n, n => n.Stack); } diff --git a/src/SMAPI/Framework/StateTracking/PlayerTracker.cs b/src/SMAPI/Framework/StateTracking/PlayerTracker.cs index c6ee12bf..5422be1d 100644 --- a/src/SMAPI/Framework/StateTracking/PlayerTracker.cs +++ b/src/SMAPI/Framework/StateTracking/PlayerTracker.cs @@ -137,7 +137,6 @@ internal class PlayerTracker : IDisposable { return this.Player.Items .Where(n => n != null) - .Distinct() .ToDictionary(n => n, n => n.Stack); } }