D17241: Disable highlighting for lines longer than 1024 characters.

[Kåre Särs]

sars updated this revision to Diff 46633.
sars retitled this revision from "WIP: Disable highlighting for lines longer than 1024 characters." to "Disable highlighting for lines longer than 1024 characters.".
sars edited the summary of this revision.
sars added a reviewer: dhaumann.
sars added a comment.


  (Remove the unrelated change in src/view/kateview.cpp)
  
  This version now disables the highlighting for lines longer than 1024 characters also when the line is selected. The selection highlight is shown tho.
  
  What is the way forward? Do we apply this? Should we modify the line length limit?
  
  I have now tested with a file with 97000 columns and I can start to see some slowdown but it is still usable.
  
  I think we could raise the limit to 50000, but also add a notification if the file contains lines longer than 1024.

REPOSITORY
  R39 KTextEditor

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D17241?vs=46632&id=46633

REVISION DETAIL
  https://phabricator.kde.org/D17241

AFFECTED FILES
  src/render/katerenderer.cpp

To: sars, cullmann, vkrause, dhaumann
Cc: brauch, kwrite-devel, kde-frameworks-devel, hase, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20181201/0cb70cd1/attachment.html>