Xargon/ES-DE
1
0
Fork 0
mirror of https://github.com/RetroDECK/ES-DE.git synced 2024-11-25 15:45:38 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Improved resilience to empty gamelist.xml files (they can now be updated instead of generating an error)

Browse source
This commit is contained in:
Leon Styhre 2024-03-09 12:24:18 +01:00
parent cd4a7d4414
commit 630ff4a0d7
1 changed files with 8 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
es-app/src/GamelistFileParser.cpp
View file

@ -135,6 +135,12 @@ namespace GamelistFileParser
return; return;
} }
if (Utils::FileSystem::getFileSize(xmlpath) == 0) {
LOG(LogWarning) << "GamelistFileParser::parseGamelist(): System \"" << system->getName()
<< "\" has an empty gamelist.xml file";
return;
}
#if defined(_WIN64) #if defined(_WIN64)
LOG(LogInfo) << "Parsing gamelist file \"" << Utils::String::replace(xmlpath, "/", "\\") LOG(LogInfo) << "Parsing gamelist file \"" << Utils::String::replace(xmlpath, "/", "\\")
<< "\"..."; << "\"...";
@ -340,7 +346,8 @@ namespace GamelistFileParser
const std::string& xmlReadPath {system->getGamelistPath(false)}; const std::string& xmlReadPath {system->getGamelistPath(false)};
bool hasAlternativeEmulatorTag {false}; bool hasAlternativeEmulatorTag {false};
if (Utils::FileSystem::exists(xmlReadPath)) { if (Utils::FileSystem::exists(xmlReadPath) &&
Utils::FileSystem::getFileSize(xmlReadPath) != 0) {
// Parse an existing file first. // Parse an existing file first.
#if defined(_WIN64) #if defined(_WIN64)