diff --git a/SabreTools.Serialization/Models/XZP/Constants.cs b/SabreTools.Serialization/Models/XZP/Constants.cs index b0e0c343..3e924ee3 100644 --- a/SabreTools.Serialization/Models/XZP/Constants.cs +++ b/SabreTools.Serialization/Models/XZP/Constants.cs @@ -14,4 +14,4 @@ namespace SabreTools.Data.Models.XZP public const uint FooterSignatureUInt32 = 0x587a4674; } -} \ No newline at end of file +} diff --git a/SabreTools.Serialization/Models/XZP/DirectoryItem.cs b/SabreTools.Serialization/Models/XZP/DirectoryItem.cs index 4f36deda..0dc5a4dc 100644 --- a/SabreTools.Serialization/Models/XZP/DirectoryItem.cs +++ b/SabreTools.Serialization/Models/XZP/DirectoryItem.cs @@ -7,7 +7,7 @@ namespace SabreTools.Data.Models.XZP public uint NameOffset { get; set; } - public string? Name { get; set; } + public string Name { get; set; } public uint TimeCreated { get; set; } } diff --git a/SabreTools.Serialization/Models/XZP/File.cs b/SabreTools.Serialization/Models/XZP/File.cs index 4e9b1063..e8268581 100644 --- a/SabreTools.Serialization/Models/XZP/File.cs +++ b/SabreTools.Serialization/Models/XZP/File.cs @@ -9,31 +9,31 @@ namespace SabreTools.Data.Models.XZP /// /// Header data /// - public Header? Header { get; set; } + public Header Header { get; set; } /// /// Directory entries data /// - public DirectoryEntry[]? DirectoryEntries { get; set; } + public DirectoryEntry[] DirectoryEntries { get; set; } /// /// Preload directory entries data /// - public DirectoryEntry[]? PreloadDirectoryEntries { get; set; } + public DirectoryEntry[] PreloadDirectoryEntries { get; set; } /// /// Preload directory mappings data /// - public DirectoryMapping[]? PreloadDirectoryMappings { get; set; } + public DirectoryMapping[] PreloadDirectoryMappings { get; set; } /// /// Directory items data /// - public DirectoryItem[]? DirectoryItems { get; set; } + public DirectoryItem[] DirectoryItems { get; set; } /// /// Footer data /// - public Footer? Footer { get; set; } + public Footer Footer { get; set; } } } diff --git a/SabreTools.Serialization/Models/XZP/Footer.cs b/SabreTools.Serialization/Models/XZP/Footer.cs index 693731c0..43b7a4fb 100644 --- a/SabreTools.Serialization/Models/XZP/Footer.cs +++ b/SabreTools.Serialization/Models/XZP/Footer.cs @@ -13,6 +13,6 @@ namespace SabreTools.Data.Models.XZP /// /// 4 bytes [MarshalAs(UnmanagedType.ByValTStr, SizeConst = 4)] - public string? Signature; + public string Signature; } } diff --git a/SabreTools.Serialization/Models/XZP/Header.cs b/SabreTools.Serialization/Models/XZP/Header.cs index 30e4fa21..76b16fb0 100644 --- a/SabreTools.Serialization/Models/XZP/Header.cs +++ b/SabreTools.Serialization/Models/XZP/Header.cs @@ -11,7 +11,7 @@ namespace SabreTools.Data.Models.XZP /// /// 4 bytes [MarshalAs(UnmanagedType.ByValTStr, SizeConst = 4)] - public string? Signature; + public string Signature; public uint Version; diff --git a/SabreTools.Serialization/Wrappers/XZP.Extraction.cs b/SabreTools.Serialization/Wrappers/XZP.Extraction.cs index 6ae8c0d7..50570f02 100644 --- a/SabreTools.Serialization/Wrappers/XZP.Extraction.cs +++ b/SabreTools.Serialization/Wrappers/XZP.Extraction.cs @@ -9,7 +9,7 @@ namespace SabreTools.Serialization.Wrappers public bool Extract(string outputDirectory, bool includeDebug) { // If we have no directory entries - if (DirectoryEntries == null || DirectoryEntries.Length == 0) + if (DirectoryEntries.Length == 0) return false; // Loop through and extract all files to the output @@ -32,11 +32,11 @@ namespace SabreTools.Serialization.Wrappers public bool ExtractFile(int index, string outputDirectory, bool includeDebug) { // If we have no directory entries - if (DirectoryEntries == null || DirectoryEntries.Length == 0) + if (DirectoryEntries.Length == 0) return false; // If we have no directory items - if (DirectoryItems == null || DirectoryItems.Length == 0) + if (DirectoryItems.Length == 0) return false; // If the directory entry index is invalid diff --git a/SabreTools.Serialization/Wrappers/XZP.Printing.cs b/SabreTools.Serialization/Wrappers/XZP.Printing.cs index 793bb631..bd977920 100644 --- a/SabreTools.Serialization/Wrappers/XZP.Printing.cs +++ b/SabreTools.Serialization/Wrappers/XZP.Printing.cs @@ -26,17 +26,10 @@ namespace SabreTools.Serialization.Wrappers Print(builder, Model.Footer); } - private static void Print(StringBuilder builder, Header? header) + private static void Print(StringBuilder builder, Header header) { builder.AppendLine(" Header Information:"); builder.AppendLine(" -------------------------"); - if (header == null) - { - builder.AppendLine(" No header"); - builder.AppendLine(); - return; - } - builder.AppendLine(header.Signature, " Signature"); builder.AppendLine(header.Version, " Version"); builder.AppendLine(header.PreloadDirectoryEntryCount, " Preload directory entry count"); @@ -49,11 +42,11 @@ namespace SabreTools.Serialization.Wrappers builder.AppendLine(); } - private static void Print(StringBuilder builder, DirectoryEntry[]? entries, string prefix) + private static void Print(StringBuilder builder, DirectoryEntry[] entries, string prefix) { builder.AppendLine($" {prefix} Entries Information:"); builder.AppendLine(" -------------------------"); - if (entries == null || entries.Length == 0) + if (entries.Length == 0) { builder.AppendLine(" No directory entries"); builder.AppendLine(); @@ -73,11 +66,11 @@ namespace SabreTools.Serialization.Wrappers builder.AppendLine(); } - private static void Print(StringBuilder builder, DirectoryMapping[]? entries) + private static void Print(StringBuilder builder, DirectoryMapping[] entries) { builder.AppendLine(" Preload Directory Mappings Information:"); builder.AppendLine(" -------------------------"); - if (entries == null || entries.Length == 0) + if (entries.Length == 0) { builder.AppendLine(" No preload directory mappings"); builder.AppendLine(); @@ -95,11 +88,11 @@ namespace SabreTools.Serialization.Wrappers builder.AppendLine(); } - private static void Print(StringBuilder builder, DirectoryItem[]? entries) + private static void Print(StringBuilder builder, DirectoryItem[] entries) { builder.AppendLine(" Directory Items Information:"); builder.AppendLine(" -------------------------"); - if (entries == null || entries.Length == 0) + if (entries.Length == 0) { builder.AppendLine(" No directory items"); builder.AppendLine(); @@ -120,17 +113,10 @@ namespace SabreTools.Serialization.Wrappers builder.AppendLine(); } - private static void Print(StringBuilder builder, Footer? footer) + private static void Print(StringBuilder builder, Footer footer) { builder.AppendLine(" Footer Information:"); builder.AppendLine(" -------------------------"); - if (footer == null) - { - builder.AppendLine(" No header"); - builder.AppendLine(); - return; - } - builder.AppendLine(footer.FileLength, " File length"); builder.AppendLine(footer.Signature, " Signature"); builder.AppendLine(); diff --git a/SabreTools.Serialization/Wrappers/XZP.cs b/SabreTools.Serialization/Wrappers/XZP.cs index 1fa411d0..13f7a766 100644 --- a/SabreTools.Serialization/Wrappers/XZP.cs +++ b/SabreTools.Serialization/Wrappers/XZP.cs @@ -14,10 +14,10 @@ namespace SabreTools.Serialization.Wrappers #region Extension Properties /// - public Data.Models.XZP.DirectoryEntry[]? DirectoryEntries => Model.DirectoryEntries; + public Data.Models.XZP.DirectoryEntry[] DirectoryEntries => Model.DirectoryEntries; /// - public Data.Models.XZP.DirectoryItem[]? DirectoryItems => Model.DirectoryItems; + public Data.Models.XZP.DirectoryItem[] DirectoryItems => Model.DirectoryItems; #endregion