flibitijibibo
commented
11 years ago Turns out the XML stuff is never used. Scrapping it, if we want to add this feature we can do it when MadTracker 3 is stable with its current features.
Closed flibitijibibo closed 11 years ago
flibitijibibo
commented
11 years ago Turns out the XML stuff is never used. Scrapping it, if we want to add this feature we can do it when MadTracker 3 is stable with its current features.
The XML stuff is only used in MTSystem, and it's a pretty small portion of the code. We could cut out a library dependency if we just use TinyXML-2:
https://github.com/leethomason/tinyxml2
Just a single cpp/h pair that can be used locally by MTSystem, and we'll just compile tinyxml2.cpp into the lib file.
This isn't hugely important, but it's one less external dependency to think about.