<table><tr><td style="">sirgienko created this revision.<br />sirgienko added a reviewer: Cantor.<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/D11588">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Octave syntax highlighter have implied, that Octave session have run before highlighter constructor had call. It worked until 17.12 version, inclusively,  but after <tt style="background: #ebebeb; font-size: 13px;">login()</tt> function for 'session' have begun to be called after the constructor, so we get a segmentation fault (call method of nullptr(m_process)). So I fix it. And I have done same small changes of debugg messages, too.</p>

<p>There is known limitation: sytax highlighting don't work before run first entry (because we call <tt style="background: #ebebeb; font-size: 13px;">login()</tt> in this time). It maybe solved, by running session after backend choosing (for getting syntax from running session directly instead pre-calculated values).</p></div></div><br /><div><strong>TEST PLAN</strong><div><ol class="remarkup-list">
<li class="remarkup-list-item">build cantor with octave_backend</li>
<li class="remarkup-list-item">run octave</li>
<li class="remarkup-list-item">check, that syntax highlighting works.</li>
</ol></div></div><br /><div><strong>REPOSITORY</strong><div><div>R55 Cantor</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11588">https://phabricator.kde.org/D11588</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/backends/octave/octavehighlighter.cpp<br />
src/backends/octave/octavesession.cpp</div></div></div><br /><div><strong>To: </strong>sirgienko, Cantor<br /><strong>Cc: </strong>Cantor, KDE Edu, narvaez, apol<br /></div>