1
0
Fork 0
forked from suyu/suyu

game_list: Resolve variable shadowing within LoadCompatibilityList()

"value" is already a used variable name within the outermost ranged-for
loop, so this variable was shadowing the outer one. This isn't a bug,
but it will get rid of a -Wshadow warning.
This commit is contained in:
Lioncash 2018-09-11 22:34:07 -04:00
parent 7fe10dea3e
commit 0e61e8362f

View file

@ -374,9 +374,9 @@ void GameList::LoadCompatibilityList() {
QString directory = game["directory"].toString(); QString directory = game["directory"].toString();
QJsonArray ids = game["releases"].toArray(); QJsonArray ids = game["releases"].toArray();
for (const QJsonValueRef& value : ids) { for (const QJsonValueRef& id_ref : ids) {
QJsonObject object = value.toObject(); QJsonObject id_object = id_ref.toObject();
QString id = object["id"].toString(); QString id = id_object["id"].toString();
compatibility_list.emplace( compatibility_list.emplace(
id.toUpper().toStdString(), id.toUpper().toStdString(),
std::make_pair(QString::number(compatibility), directory)); std::make_pair(QString::number(compatibility), directory));