[OfflineList] Use new parser, update TODO

This commit is contained in:
Matt Nadareski
2018-01-17 00:25:26 -08:00
parent 8d5310bfa0
commit b007868621

View File

@@ -24,7 +24,7 @@ namespace SabreTools.Library.DatFiles
/// <summary> /// <summary>
/// Represents parsing and writing of an OfflineList XML DAT /// Represents parsing and writing of an OfflineList XML DAT
/// </summary> /// </summary>
/// TODO: Verify that all read/write for this DatFile type is correct /// TODO: Verify that all write for this DatFile type is correct
internal class OfflineList : DatFile internal class OfflineList : DatFile
{ {
/// <summary> /// <summary>
@@ -58,10 +58,6 @@ namespace SabreTools.Library.DatFiles
bool clean, bool clean,
bool remUnicode) bool remUnicode)
{ {
// All XML-derived DATs share a lot in common so it just calls one implementation
new Logiqx(this, false).ParseFile(filename, sysid, srcid, keep, clean, remUnicode);
return;
Encoding enc = Utilities.GetEncoding(filename); Encoding enc = Utilities.GetEncoding(filename);
XmlReader xtr = Utilities.GetXmlTextReader(filename); XmlReader xtr = Utilities.GetXmlTextReader(filename);
@@ -563,7 +559,7 @@ namespace SabreTools.Library.DatFiles
bool remUnicode) bool remUnicode)
{ {
// Prepare all internal variables // Prepare all internal variables
string releaseNumber = "", key = "", publisher = ""; string releaseNumber = "", key = "", publisher = "", duplicateid = "";
long size = -1; long size = -1;
List<Rom> roms = new List<Rom>(); List<Rom> roms = new List<Rom>();
Machine machine = new Machine(); Machine machine = new Machine();
@@ -657,7 +653,11 @@ namespace SabreTools.Library.DatFiles
break; break;
case "duplicateid": case "duplicateid":
machine.CloneOf = reader.ReadElementContentAsString(); duplicateid = reader.ReadElementContentAsString();
if (duplicateid != "0")
{
machine.CloneOf = duplicateid;
}
break; break;
default: default: