<table><tr><td style="">sirgienko created this revision.<br />sirgienko added reviewers: Cantor, asemke.<br />sirgienko added a project: Cantor.<br />Restricted Application added a subscriber: KDE Edu.<br />Restricted Application added a project: KDE Edu.<br />sirgienko 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/D12565">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Before, when we work with octave and use 'Restart Backend' button, we suddenly get foreign output. So this commit fix it by adding backend cleanup of inner state in logout. Also the commit move <tt style="background: #ebebeb; font-size: 13px;">emit loginDone</tt> statement, because we emit this signal <strong>before</strong> we real finish preparation.</p></div></div><br /><div><strong>TEST PLAN</strong><div><ol class="remarkup-list">
<li class="remarkup-list-item">Run octave backend without patch</li>
<li class="remarkup-list-item">Write some entries and run they</li>
<li class="remarkup-list-item">Reboot octave backend by the button and check, that the output changes</li>
<li class="remarkup-list-item">Apply patch</li>
<li class="remarkup-list-item">Check, that now all fine.</li>
</ol></div></div><br /><div><strong>REPOSITORY</strong><div><div>R55 Cantor</div></div></div><br /><div><strong>BRANCH</strong><div><div>improve-octave-login</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12565">https://phabricator.kde.org/D12565</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/backends/octave/octavesession.cpp</div></div></div><br /><div><strong>To: </strong>sirgienko, Cantor, asemke<br /><strong>Cc: </strong>Cantor, KDE Edu, narvaez, apol<br /></div>