From 397f8019a0cd83c2a0b04c3b9657f35088cdb561 Mon Sep 17 00:00:00 2001 From: Matt Nadareski Date: Sun, 30 Jun 2024 00:02:53 -0400 Subject: [PATCH] Remove unnecessary check in rebuilding --- SabreTools.DatTools/Rebuilder.cs | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/SabreTools.DatTools/Rebuilder.cs b/SabreTools.DatTools/Rebuilder.cs index 01bb6510..4c7b21bc 100644 --- a/SabreTools.DatTools/Rebuilder.cs +++ b/SabreTools.DatTools/Rebuilder.cs @@ -438,11 +438,12 @@ namespace SabreTools.DatTools foreach (DatItem item in dupes) { // If we don't have a proper machine - if (item.GetFieldValue(DatItem.MachineKey)!?.GetStringFieldValue(Models.Metadata.Machine.NameKey) == null || !datFile.Items.ContainsKey(item.GetFieldValue(DatItem.MachineKey)!.GetStringFieldValue(Models.Metadata.Machine.NameKey)!)) + var machine = item.GetFieldValue(DatItem.MachineKey); + if (machine?.GetStringFieldValue(Models.Metadata.Machine.NameKey) == null) continue; // If we should check for the items in the machine - var items = datFile.Items[item.GetFieldValue(DatItem.MachineKey)!.GetStringFieldValue(Models.Metadata.Machine.NameKey)!]; + var items = datFile.Items[machine.GetStringFieldValue(Models.Metadata.Machine.NameKey)!]; if (shouldCheck && items!.Count > 1) outputFormat = OutputFormat.Folder; else if (shouldCheck && items!.Count == 1)