D17241: WIP:Disable highlighting after 512 characters on a line.

Milian Wolff noreply at phabricator.kde.org
Mon Jun 17 22:57:59 BST 2019

mwolff added a comment.

  I personally dislike this too.


> katerenderer.cpp:408
> +        auto &currentRange = renderRanges.pushNewRange();
>          for (int i = 0; i < al.count(); ++i)
>              if (al[i].length > 0 && al[i].attributeValue > 0) {

couldn't the al.count be limited here? if I understand the patch correctly, we throw out all highlighting if we encounter too many ranges. but could we instead just skip the items that are beyond the limit?

  R39 KTextEditor


To: cullmann, vkrause, dhaumann, mwolff, sars
Cc: zetazeta, mwolff, brauch, kwrite-devel, kde-frameworks-devel, LeGast00n, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20190617/201239df/attachment.html>

More information about the KWrite-Devel mailing list