Xargon/ES-DE
1
0
Fork 0
mirror of https://github.com/RetroDECK/ES-DE.git synced 2025-04-10 19:15:13 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Duplicate es_systems.xml command label entries are now ignored on application startup

Browse source
This commit is contained in:
Leon Styhre 2023-10-10 17:21:10 +02:00
parent 8efc5e9f22
commit cd3f56af77
1 changed files with 15 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
es-app/src/SystemData.cpp
View file

@ -682,8 +682,21 @@ bool SystemData::loadConfig()
<< name << "\"";
break;
}
commands.emplace_back(
std::make_pair(entry.text().get(), entry.attribute("label").as_string()));
// Skip any duplicate entries (i.e. those with identical labels).
bool duplicateLabel {false};
for (auto& command : commands) {
if (command.second == entry.attribute("label").as_string()) {
LOG(LogError)
<< "Duplicate command label \"" << entry.attribute("label").as_string()
<< "\" defined for system \"" << name << "\", ignoring entry";
duplicateLabel = true;
break;
}
}
if (!duplicateLabel) {
commands.emplace_back(
std::make_pair(entry.text().get(), entry.attribute("label").as_string()));
}
}
// Platform ID list