<table><tr><td style="">sars updated this revision to Diff 46633.<br />sars retitled this revision from "WIP: Disable highlighting for lines longer than 1024 characters." to "Disable highlighting for lines longer than 1024 characters.".<br />sars edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-w75vkhwhfjx5xgh/">(Show Details)</a><br />sars added a reviewer: dhaumann.<br />sars 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>(Remove the unrelated change in src/view/kateview.cpp)</p>

<p>This version now disables the highlighting for lines longer than 1024 characters also when the line is selected. The selection highlight is shown tho.</p>

<p>What is the way forward? Do we apply this? Should we modify the line length limit?</p>

<p>I have now tested with a file with 97000 columns and I can start to see some slowdown but it is still usable.</p>

<p>I think we could raise the limit to 50000, but also add a notification if the file contains lines longer than 1024.</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">A lot of time is spent in decorationsForLine() when we have long lines<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">. I noticed that we iterate the same line more than once for every edit, but I have not figured out where we could delay or remove iterations (yet ?)</span>.<br />
<br />
Disabling all highlighting for lines longer than 1024 improves the rendering speed very much, but again it is not so nice to not have highlighting.<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"><br />
I also noticed that this patch is not good enough since if I select all of the long line the line will be highlighted...</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R39 KTextEditor</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D17241?vs=46632&id=46633">https://phabricator.kde.org/D17241?vs=46632&id=46633</a></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>AFFECTED FILES</strong><div><div>src/render/katerenderer.cpp</div></div></div><br /><div><strong>To: </strong>sars, cullmann, vkrause, dhaumann<br /><strong>Cc: </strong>brauch, kwrite-devel, kde-frameworks-devel, hase, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann<br /></div>