<table><tr><td style="">alex created this revision.<br />alex added reviewers: davidedmundson, ngraham, broulik, meven.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />alex requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D27513">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Duplicate function calls to load the config in the runner/config have been removed.<br />
Additionally the config keys/sorting enum have been moved to a new file, before this the runner included the header of the config class.<br />
The actions of the runner are also refactored, similar to <a href="https://phabricator.kde.org/D3688" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D3688</a>.<br />
Also the user (owner of process) has been removed from the subtext, because the second line is not shown and adding the user info to the same line would be too long.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Should compile and kill processes as before.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>BRANCH</strong><div><div>killrunner_cleanup (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27513">https://phabricator.kde.org/D27513</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>runners/kill/CMakeLists.txt<br />
runners/kill/config_keys.h<br />
runners/kill/killrunner.cpp<br />
runners/kill/killrunner.h<br />
runners/kill/killrunner_config.cpp<br />
runners/kill/killrunner_config.h</div></div></div><br /><div><strong>To: </strong>alex, davidedmundson, ngraham, broulik, meven<br /><strong>Cc: </strong>plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>