<table><tr><td style="">dfaure created this revision.<br />dfaure added reviewers: sandsmark, mlaurent.<br />Restricted Application added a project: KDE PIM.<br />Restricted Application edited subscribers, added: kde-pim; removed: KDE PIM.<br />dfaure 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/D14448">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>With the sonnet fix <a href="https://phabricator.kde.org/D14447" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D14447</a>,<br />
setLanguage takes care of scheduling a call to rehighlight (via a timer<br />
which is already started in the constructor anyway),<br />
so that we don't have to call it directly here, leading to two calls.</p>

<p>The hackish processEvents() has to be removed too, since otherwise it<br />
would process the ctor timer and rehighlight before the call to<br />
setLanguage even happens. I think the 500ms timer below that<br />
processEvents() call means that we don't need it anyway.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Replying to a mail in kmail, and checking how many times<br />
slotRehighlight is called. With and without autodetection of language.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R86 PIM: Text Editor</div></div></div><br /><div><strong>BRANCH</strong><div><div>Applications/18.04</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14448">https://phabricator.kde.org/D14448</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/composer-ng/richtextcomposercontroler.cpp<br />
src/texteditor/richtexteditor/richtexteditor.cpp</div></div></div><br /><div><strong>To: </strong>dfaure, sandsmark, mlaurent<br /><strong>Cc: </strong>kde-pim, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>