D27433: Check activeModule before using it

2020-02-16 Thread Wolfgang Bauer
wbauer closed this revision. wbauer added a comment. Committed with https://commits.kde.org/kcmutils/ea7120ed901bf6161bb483ab73211a6491daac8f REPOSITORY R295 KCMUtils REVISION DETAIL https://phabricator.kde.org/D27433 To: wbauer, #frameworks, davidedmundson, ervin Cc: kde-frameworks-dev

D27433: Check activeModule before using it

2020-02-16 Thread Kevin Ottens
ervin accepted this revision. REPOSITORY R295 KCMUtils REVISION DETAIL https://phabricator.kde.org/D27433 To: wbauer, #frameworks, davidedmundson, ervin Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27433: Check activeModule before using it

2020-02-16 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R295 KCMUtils REVISION DETAIL https://phabricator.kde.org/D27433 To: wbauer, #frameworks, davidedmundson Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27433: Check activeModule before using it

2020-02-16 Thread Wolfgang Bauer
wbauer edited the summary of this revision. REPOSITORY R295 KCMUtils REVISION DETAIL https://phabricator.kde.org/D27433 To: wbauer, #frameworks Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27433: Check activeModule before using it

2020-02-16 Thread Wolfgang Bauer
wbauer created this revision. wbauer added a reviewer: Frameworks. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. wbauer requested review of this revision. REVISION SUMMARY `activeModule` can be a nullptr here, as this is outside/after the `if (activeModule