Reduce redirection for Name setters and getters

This commit is contained in:
Matt Nadareski
2026-04-04 10:25:49 -04:00
parent 845a0cb998
commit 872fdc2a10
44 changed files with 434 additions and 4 deletions

View File

@@ -1,7 +1,6 @@
using System;
using System.Xml.Serialization;
using Newtonsoft.Json;
using SabreTools.Data.Extensions;
using SabreTools.Hashing;
using SabreTools.Logging;
using SabreTools.Metadata.DatItems.Formats;
@@ -101,13 +100,13 @@ namespace SabreTools.Metadata.DatItems
/// Gets the name to use for a DatItem
/// </summary>
/// <returns>Name if available, null otherwise</returns>
public virtual string? GetName() => _internal.GetName();
public abstract string? GetName();
/// <summary>
/// Sets the name to use for a DatItem
/// </summary>
/// <param name="name">Name to set for the item</param>
public virtual void SetName(string? name) => _internal.SetName(name);
public abstract void SetName(string? name);
#endregion

View File

@@ -54,6 +54,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -38,6 +38,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => null;
/// <inheritdoc/>
public override void SetName(string? name) { }
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -229,6 +229,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -50,6 +50,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -32,6 +32,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => null;
/// <inheritdoc/>
public override void SetName(string? name) { }
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -68,6 +68,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -56,6 +56,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => null;
/// <inheritdoc/>
public override void SetName(string? name) { }
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -50,6 +50,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -60,6 +60,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -77,6 +77,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -104,6 +104,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => null;
/// <inheritdoc/>
public override void SetName(string? name) { }
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -57,6 +57,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -81,6 +81,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => null;
/// <inheritdoc/>
public override void SetName(string? name) { }
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -38,6 +38,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -50,6 +50,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -104,6 +104,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -60,6 +60,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -126,6 +126,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -39,6 +39,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -145,6 +145,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => null;
/// <inheritdoc/>
public override void SetName(string? name) { }
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -104,6 +104,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => null;
/// <inheritdoc/>
public override void SetName(string? name) { }
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -38,6 +38,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -62,6 +62,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -7,6 +7,7 @@ using SabreTools.IO.Extensions;
using SabreTools.Text.Extensions;
// TODO: Add item mappings for all fields
// TODO: Investigate whether this is the "File" from NIDB
namespace SabreTools.Metadata.DatItems.Formats
{
/// <summary>
@@ -104,6 +105,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => null;
/// <inheritdoc/>
public override void SetName(string? name) { }
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -44,6 +44,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -79,6 +79,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => null;
/// <inheritdoc/>
public override void SetName(string? name) { }
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -44,6 +44,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -79,6 +79,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -56,6 +56,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -71,6 +71,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -49,6 +49,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => null;
/// <inheritdoc/>
public override void SetName(string? name) { }
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -50,6 +50,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -62,6 +62,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -127,6 +127,16 @@ namespace SabreTools.Metadata.DatItems.Formats
}
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => null;
/// <inheritdoc/>
public override void SetName(string? name) { }
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -401,6 +401,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -38,6 +38,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -116,6 +116,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => null;
/// <inheritdoc/>
public override void SetName(string? name) { }
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -44,6 +44,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -38,7 +38,7 @@ namespace SabreTools.Metadata.DatItems.Formats
{
// Handle subitems
if (item.SlotOption is not null)
SlotOption = Array.ConvertAll(item.SlotOption, slotOption => new SlotOption(slotOption));;
SlotOption = Array.ConvertAll(item.SlotOption, slotOption => new SlotOption(slotOption)); ;
}
public Slot(Data.Models.Metadata.Slot item, Machine machine, Source source) : this(item)
@@ -49,6 +49,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -50,6 +50,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -56,6 +56,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => Name;
/// <inheritdoc/>
public override void SetName(string? name) => Name = name;
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -38,6 +38,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => null;
/// <inheritdoc/>
public override void SetName(string? name) { }
#endregion
#region Cloning Methods
/// <inheritdoc/>

View File

@@ -176,6 +176,16 @@ namespace SabreTools.Metadata.DatItems.Formats
#endregion
#region Accessors
/// <inheritdoc/>
public override string? GetName() => null;
/// <inheritdoc/>
public override void SetName(string? name) { }
#endregion
#region Cloning Methods
/// <inheritdoc/>