Consolidate RemoveItemsFromRomOfChild invocation

This commit is contained in:
Matt Nadareski
2025-01-13 14:34:15 -05:00
parent 2d277607cf
commit 73b4d9650a
2 changed files with 10 additions and 12 deletions

View File

@@ -79,12 +79,14 @@ namespace SabreTools.DatFiles
/// <summary>
/// Use romof tags to remove bios items from children
/// </summary>
/// <param name="bios">True if only child Bios sets are touched, false for non-bios sets</param>
/// <remarks>Assumes items are bucketed by <see cref="ItemKey.Machine"/></remarks>
public void RemoveItemsFromRomOfChild(bool bios)
public void RemoveItemsFromRomOfChild()
{
RemoveItemsFromRomOfChildImpl(bios);
RemoveItemsFromRomOfChildImplDB(bios);
// TODO: Figure out why the bios flag is needed
RemoveItemsFromRomOfChildImpl(false);
RemoveItemsFromRomOfChildImplDB(false);
RemoveItemsFromRomOfChildImpl(true);
RemoveItemsFromRomOfChildImplDB(true);
}
/// <summary>

View File

@@ -127,8 +127,7 @@ namespace SabreTools.DatTools
datFile.AddItemsFromChildren(true, false);
// Now that we have looped through the cloneof tags, we loop through the romof tags
datFile.RemoveItemsFromRomOfChild(false);
datFile.RemoveItemsFromRomOfChild(true);
datFile.RemoveItemsFromRomOfChild();
// Finally, remove the romof and cloneof tags so it's not picked up by the manager
datFile.RemoveMachineRelationshipTags();
@@ -172,8 +171,7 @@ namespace SabreTools.DatTools
datFile.AddItemsFromChildren(true, true);
// Now that we have looped through the cloneof tags, we loop through the romof tags
datFile.RemoveItemsFromRomOfChild(false);
datFile.RemoveItemsFromRomOfChild(true);
datFile.RemoveItemsFromRomOfChild();
// Finally, remove the romof and cloneof tags so it's not picked up by the manager
datFile.RemoveMachineRelationshipTags();
@@ -194,8 +192,7 @@ namespace SabreTools.DatTools
datFile.AddItemsFromCloneOfParent();
// Now that we have looped through the cloneof tags, we loop through the romof tags
datFile.RemoveItemsFromRomOfChild(false);
datFile.RemoveItemsFromRomOfChild(true);
datFile.RemoveItemsFromRomOfChild();
// Finally, remove the romof and cloneof tags so it's not picked up by the manager
datFile.RemoveMachineRelationshipTags();
@@ -216,8 +213,7 @@ namespace SabreTools.DatTools
datFile.RemoveItemsFromCloneOfChild();
// Now that we have looped through the cloneof tags, we loop through the romof tags
datFile.RemoveItemsFromRomOfChild(false);
datFile.RemoveItemsFromRomOfChild(true);
datFile.RemoveItemsFromRomOfChild();
// Finally, remove the romof and cloneof tags so it's not picked up by the manager
datFile.RemoveMachineRelationshipTags();