claunia/SabreTools
1
0
Fork 0
mirror of https://github.com/claunia/SabreTools.git synced 2025-12-16 19:14:27 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix ClrMamePro writer issues

Browse Source
This commit is contained in:
Matt Nadareski
2023-07-28 22:52:10 -04:00
parent 54393a8982
commit 9e773b39c3
1 changed files with 10 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
SabreTools.DatFiles/Formats/ClrMamePro.Writer.cs
View File

@@ -141,7 +141,7 @@ namespace SabreTools.DatFiles.Formats
{ {
logger.User($"Writing to '{outfile}'..."); logger.User($"Writing to '{outfile}'...");
var metadataFile = CreateMetadataFile(); var metadataFile = CreateMetadataFile(ignoreblanks);
if (!Serialization.ClrMamePro.SerializeToFile(metadataFile, outfile, Quotes)) if (!Serialization.ClrMamePro.SerializeToFile(metadataFile, outfile, Quotes))
{ {
logger.Warning($"File '{outfile}' could not be written! See the log for more details."); logger.Warning($"File '{outfile}' could not be written! See the log for more details.");
@@ -160,12 +160,13 @@ namespace SabreTools.DatFiles.Formats
/// <summary> /// <summary>
/// Create a MetadataFile from the current internal information /// Create a MetadataFile from the current internal information
/// <summary> /// <summary>
private Models.ClrMamePro.MetadataFile CreateMetadataFile() /// <param name="ignoreblanks">True if blank roms should be skipped on output, false otherwise</param>
private Models.ClrMamePro.MetadataFile CreateMetadataFile(bool ignoreblanks)
{ {
var metadataFile = new Models.ClrMamePro.MetadataFile var metadataFile = new Models.ClrMamePro.MetadataFile
{ {
ClrMamePro = CreateClrMamePro(), ClrMamePro = CreateClrMamePro(),
Game = CreateGames() Game = CreateGames(ignoreblanks)
}; };
return metadataFile; return metadataFile;
} }
@@ -206,7 +207,8 @@ namespace SabreTools.DatFiles.Formats
/// <summary> /// <summary>
/// Create an array of GameBase from the current internal information /// Create an array of GameBase from the current internal information
/// <summary> /// <summary>
private Models.ClrMamePro.GameBase[]? CreateGames() /// <param name="ignoreblanks">True if blank roms should be skipped on output, false otherwise</param>
private Models.ClrMamePro.GameBase[]? CreateGames(bool ignoreblanks)
{ {
// If we don't have items, we can't do anything // If we don't have items, we can't do anything
if (this.Items == null || !this.Items.Any()) if (this.Items == null || !this.Items.Any())
@@ -252,6 +254,10 @@ namespace SabreTools.DatFiles.Formats
// Loop through and convert the items to respective lists // Loop through and convert the items to respective lists
foreach (var item in items) foreach (var item in items)
{ {
// Skip if we're ignoring the item
if (ShouldIgnore(item, ignoreblanks))
continue;
switch (item) switch (item)
{ {
case Release release: case Release release: