SabreTools/SabreTools.Serialization
1
0
Fork 0
mirror of https://github.com/SabreTools/SabreTools.Serialization.git synced 2026-07-09 02:16:55 +00:00
Code Issues 56 Packages Projects Releases 20 Wiki Activity

Remove items when machine is removed for DB

Browse Source
This commit is contained in:
Matt Nadareski
2026-04-16 17:00:16 -04:00
parent 0feb4130fe
commit 2caab2c9cf
1 changed files with 22 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
SabreTools.Metadata.DatFiles/ItemDatabase.cs
View File

@@ -471,12 +471,29 @@ namespace SabreTools.Metadata.DatFiles
_machines.Remove(machineIndex);
#endif
// TODO: Figure out if removing or marking items makes sense here
// Get the current list of item indicies
long[] itemIndicies = [.. _items.Keys];
// The old version of this code removed all items that were associated
// with the machine. However, only items with a machine will ever be
// written out. The extra processing time to find and remove all
// of the related items seems unnecessary with this in mind.
#if NET40_OR_GREATER || NETCOREAPP || NETSTANDARD2_0_OR_GREATER
Parallel.For(0, itemIndicies.Length, i =>
#else
for (int i = 0; i < itemIndicies.Length; i++)
#endif
{
var datItem = GetItem(i);
if (datItem is null || datItem.MachineIndex != machineIndex)
#if NET40_OR_GREATER || NETCOREAPP || NETSTANDARD2_0_OR_GREATER
return;
#else
continue;
#endif
RemoveItem(i);
#if NET40_OR_GREATER || NETCOREAPP || NETSTANDARD2_0_OR_GREATER
});
#else
}
#endif
return true;
}