<table><tr><td style="">dhaumann 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/D17241">View Revision</a></tr></table><br /><div><div><p>Is it correct that highlighting in the document (i.e. KSyntaxHighlighting) still takes place for the entire line, and this change simply only changes the fact that KateRenderer stops using the highlighting info after 100000 columns?</p>

<p>Prior to accepting this patch, would it be possible to fix this differently, i.e. by profiling and fixing this properly instead of applying a workaround?</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D17241#inline-95548">View Inline</a><span style="color: #4b4d51; font-weight: bold;">katetextbuffer.cpp:71</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; ">    <span class="p">,</span> <span class="n">m_newLineAtEof</span><span class="p">(</span><span style="color: #304a96">false</span><span class="p">)</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(251, 175, 175, .7);">    <span class="p">,</span> <span class="n">m_lineLengthLimit</span><span class="p">(<span class="bright"></span></span><span class="bright"></span><span style="color: #601200"><span class="bright">4096</span></span><span class="p">)</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span class="p">,</span> <span class="n">m_lineLengthLimit</span><span class="p">(<span class="bright"></span></span><span class="bright"></span><span style="color: #601200"><span class="bright">100000</span></span><span class="p">)</span>
</div><div style="padding: 0 8px; margin: 0 4px; ">    <span class="p">,</span> <span class="n">m_alwaysUseKAuthForSave</span><span class="p">(</span><span class="n">alwaysUseKAuth</span><span class="p">)</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Hm, isn't the line length limit configurable in kateconfig.cpp? There you will find a key called</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">const char KEY_LINE_LENGTH_LIMIT[] = "Line Length Limit";</pre></div>

<p style="padding: 0; margin: 8px;">that is used to read/write the config file.</p>

<p style="padding: 0; margin: 8px;">I think if this is changed here to 100000, it should be changed to the same value in KateDocumentConfig::readValue():</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">setLineLengthLimit(config.readEntry(KEY_LINE_LENGTH_LIMIT, 4096));</pre></div>

<p style="padding: 0; margin: 8px;">And I would even suggest to rename as follows:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">const char KEY_LINE_LENGTH_LIMIT[] = "Line Length Limit"; + const char KEY_LINE_LENGTH_LIMIT[] = "Line Length Limitation";</li>
</ul>

<p style="padding: 0; margin: 8px;">Since then, all users will automatically get a reset here. Without this change, all users will stay on the 4096 limit.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R39 KTextEditor</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17241">https://phabricator.kde.org/D17241</a></div></div><br /><div><strong>To: </strong>sars, cullmann, vkrause, dhaumann, mwolff<br /><strong>Cc: </strong>mwolff, brauch, kwrite-devel, kde-frameworks-devel, hase, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann<br /></div>