mirror of
https://github.com/claunia/SabreTools.git
synced 2025-12-16 19:14:27 +00:00
Make TrimMerge and DatSplit properly bool-returning
This commit is contained in:
@@ -35,14 +35,19 @@ namespace SabreTools
|
||||
/// <summary>
|
||||
/// Trim and process the given DAT or folder of DATs
|
||||
/// </summary>
|
||||
public void Process()
|
||||
/// <returns>True if the DAT could be updated, false otherwise</returns>
|
||||
public bool Process()
|
||||
{
|
||||
// If file doesn't exist, error and return
|
||||
if (!File.Exists(_filename))
|
||||
{
|
||||
_logger.Error("File '" + _filename + "' doesn't exist");
|
||||
return false;
|
||||
}
|
||||
|
||||
_path = (_path == "" ? Environment.CurrentDirectory : _path);
|
||||
|
||||
// Drag and drop means quotes; we don't want quotes
|
||||
_filename = _filename.Replace("\"", "");
|
||||
|
||||
// We also want the full path of the file, just in case
|
||||
// We want the full path of the file, just in case
|
||||
_filename = Path.GetFullPath(_filename);
|
||||
|
||||
// If it's a single file, handle it as such
|
||||
@@ -68,7 +73,7 @@ namespace SabreTools
|
||||
}
|
||||
}
|
||||
|
||||
_logger.Close();
|
||||
return true;
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
|
||||
@@ -68,14 +68,28 @@ namespace SabreTools
|
||||
/// <summary>
|
||||
/// Split a DAT based on filtering by 2 extensions
|
||||
/// </summary>
|
||||
public void Split()
|
||||
/// <returns>True if DAT was split, false otherwise</returns>
|
||||
public bool Split()
|
||||
{
|
||||
// If file doesn't exist, error and return
|
||||
if (!File.Exists(_filename))
|
||||
{
|
||||
_logger.Error("File '" + _filename + "' doesn't exist");
|
||||
return false;
|
||||
}
|
||||
|
||||
List<RomData> romsA = new List<RomData>();
|
||||
List<RomData> romsB = new List<RomData>();
|
||||
|
||||
// Load the current DAT to be processed
|
||||
List<RomData> roms = RomManipulation.Parse(_filename, 0, 0, _logger);
|
||||
|
||||
// If roms is empty, return false
|
||||
if (roms.Count == 0)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
// Now separate the roms accordingly
|
||||
foreach (RomData rom in roms)
|
||||
{
|
||||
@@ -95,10 +109,12 @@ namespace SabreTools
|
||||
}
|
||||
|
||||
// Then write out both files
|
||||
Output.WriteToDat(Path.GetFileNameWithoutExtension(_filename) + "." + _extA, Path.GetFileNameWithoutExtension(_filename) + "." + _extA,
|
||||
bool success = Output.WriteToDat(Path.GetFileNameWithoutExtension(_filename) + "." + _extA, Path.GetFileNameWithoutExtension(_filename) + "." + _extA,
|
||||
"", "", "", "", false, !RomManipulation.IsXmlDat(_filename), "", romsA, _logger);
|
||||
Output.WriteToDat(Path.GetFileNameWithoutExtension(_filename) + "." + _extB, Path.GetFileNameWithoutExtension(_filename) + "." + _extB,
|
||||
success &= Output.WriteToDat(Path.GetFileNameWithoutExtension(_filename) + "." + _extB, Path.GetFileNameWithoutExtension(_filename) + "." + _extB,
|
||||
"", "", "", "", false, !RomManipulation.IsXmlDat(_filename), "", romsB, _logger);
|
||||
|
||||
return success;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user