jyapayne/moonlight-qt
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Avoid asserting if we get a box art load completion callback for an app that's gone

Browse source
This commit is contained in:
Cameron Gutman 2018-07-26 22:16:27 -07:00
commit eb1316ce32
1 changed files with 10 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

15
app/gui/appmodel.cpp
View file

@ -123,10 +123,15 @@ void AppModel::handleBoxArtLoaded(NvComputer* computer, NvApp app, QUrl /* image
Q_ASSERT(computer == m_Computer); Q_ASSERT(computer == m_Computer);
int index = m_Apps.indexOf(app); int index = m_Apps.indexOf(app);
Q_ASSERT(index >= 0);
// Let our view know the box art data has changed for this app // Make sure we're not delivering a callback to an app that's already been removed
emit dataChanged(createIndex(index, 0), if (index >= 0) {
createIndex(index, 0), // Let our view know the box art data has changed for this app
QVector<int>() << BoxArtRole); emit dataChanged(createIndex(index, 0),
createIndex(index, 0),
QVector<int>() << BoxArtRole);
}
else {
qWarning() << "App not found for box art callback:" << app.name;
}
} }