<table><tr><td style="">loh.tar 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/D18125">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>m_dictionaryGroup->checkedAction() might be null.</p></blockquote>

<p>Oops! Thanks.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>For the naming: Given we show the encodings with their short-names, too, I can live with de_DE or stuff like that.</p></blockquote>

<p>Is it OK to add some cutting like that? Currently is that extra stuff shown</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">en-wo_accents -> en</li>
<li class="remarkup-list-item">en_AU-variant_0 -> en_AU</li>
</ul>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>For the overhead: I would like to have this optional and disabled per default, that avoids all that work in cursorPositionChanged()<br />
 Just like we do for word count/...</p></blockquote>

<p>Hm, would be nice I could do this in an extra patch. At best after my "general config interface idea" (where I have nothing tried to do as of now)<br />
Did you do some perf tests or simiar, is it so much extra work?<br />
Ideas to reduce the overhead?</p></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18125">https://phabricator.kde.org/D18125</a></div></div><br /><div><strong>To: </strong>loh.tar, KTextEditor, cullmann<br /><strong>Cc: </strong>cullmann, ngraham, dhaumann, kwrite-devel, kde-frameworks-devel, KTextEditor, hase, michaelh, bruns, demsking, sars<br /></div>