yukijoou/kwin
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Make sure that the enableWindowManagement and handleWindowManagement

Browse source 
will not crash
This commit is contained in:
Antonis Tsiapaliokas 2013-09-10 13:38:05 +03:00 committed by Martin Gräßlin
parent 50dd943e7d
commit 12f8544b0f
1 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
kcmkwin/kwincompositing/model.cpp
View file

@ -210,7 +210,8 @@ void EffectModel::handleDesktopSwitching(int row)
void EffectModel::handleWindowManagement(int row, bool enabled) void EffectModel::handleWindowManagement(int row, bool enabled)
{ {
if (m_effectsList.size() > 0) //Make sure that our row is valid
if (m_effectsList.size() > 0 && row < = m_effectsList.size())
m_effectsList[row].effectStatus = enabled; m_effectsList[row].effectStatus = enabled;
} }
@ -267,6 +268,10 @@ void EffectModel::syncConfig()
void EffectModel::enableWidnowManagement(bool enabled) void EffectModel::enableWidnowManagement(bool enabled)
{ {
//Make sure that our effects are being installed properly
if (m_effectsList.size() <= 0)
return;
int desktopGridRow = findRowByServiceName("kwin4_effect_desktopgrid"); int desktopGridRow = findRowByServiceName("kwin4_effect_desktopgrid");
const QModelIndex desktopGridIndex = createIndex(desktopGridRow, 0); const QModelIndex desktopGridIndex = createIndex(desktopGridRow, 0);
setData(desktopGridIndex, enabled, EffectModel::WindowManagementRole); setData(desktopGridIndex, enabled, EffectModel::WindowManagementRole);