davidedmundson accepted this revision. davidedmundson added inline comments. This revision is now accepted and ready to land.
INLINE COMMENTS > systemmodel.cpp:57 > { > - qDeleteAll(m_entryList); > -} > - > -void SystemModel::init() > -{ > - QList<SystemEntry *> actions; > - > - actions << new SystemEntry(this, SystemEntry::LockSession); > - actions << new SystemEntry(this, SystemEntry::LogoutSession); > - actions << new SystemEntry(this, SystemEntry::SaveSession); > - actions << new SystemEntry(this, SystemEntry::SwitchUser); > - actions << new SystemEntry(this, SystemEntry::SuspendToRam); > - actions << new SystemEntry(this, SystemEntry::SuspendToDisk); > - actions << new SystemEntry(this, SystemEntry::Reboot); > - actions << new SystemEntry(this, SystemEntry::Shutdown); > - > - foreach(SystemEntry *entry, actions) { > - if (entry->isValid()) { > - m_entryList << entry; > - } else { > - delete entry; > - } > - } > + qDeleteAll(m_entries.values()); > } qDeleteAll(m_entries) REPOSITORY R119 Plasma Desktop BRANCH master REVISION DETAIL https://phabricator.kde.org/D20237 To: hein, #plasma, davidedmundson Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart