<table><tr><td style="">asemke 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/D12504">View Revision</a></tr></table><br /><div><div><p>I cannot reproduce the problem with the first expression never ending. Which expression do you use?</p>

<p>Also here, the behavior and the implementation is inconsistent for different backends. For some backends like maxima we have a hard coded list of keywords, for others we fetch this list from the backend. In order to make the syntax highlighting and completion possible we need to always login for such backends in Worksheet::showCompletion() which breaks the whole idea of postponing the login. With your patch you won't have, if I see it correctly, no syntax highlighting and completion until the login is done. So, for saved scilab projects and for new projects where there user wants to write down some code first and evaluate later there will be no syntax highlighting and completion.</p>

<p>I'd rather also re-factor this part in Cantor. The syntax highlighting and completion should be implemented with the help of KSyntaxHighlighting as proposed in <a href="https://phabricator.kde.org/T5382" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T5382</a>. Once this is implemented we can remove a lot of code in Cantor and also really do the login if it's really required for evaluations.</p></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/D12504">https://phabricator.kde.org/D12504</a></div></div><br /><div><strong>To: </strong>sirgienko, Cantor, asemke<br /><strong>Cc: </strong>Cantor, KDE Edu, narvaez, apol<br /></div>