<table><tr><td style="">romangg added a comment.
</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/D16277">View Revision</a></tr></table><br /><div><div><p>The functional split is kind of confusing. Some functions in logout.cpp still do shutdown related tasks when looking at their names, other in shutdown.cpp do logout things.</p>

<p>To me this whole distinction between shutdown and logout seems kind of artificial. In the end the main function is that the session quits, If afterwards the device shuts down or we go back to the login maanger should be only a side note.</p>

<p>Is my observation wrong? Or do you plan on working in this direction in the future?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16277">https://phabricator.kde.org/D16277</a></div></div><br /><div><strong>To: </strong>davidedmundson, Plasma, apol<br /><strong>Cc: </strong>romangg, apol, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, mart<br /></div>