cullmann accepted this revision.
cullmann added a comment.
This revision is now accepted and ready to land.


  I think that lookAhead rules don't publish their length sounds like a sane 
improvement.
  
  All in all, I think the folding still has some general issues in how we 
handle it.
  
  e.g. I have ATM "interesting" highlighted areas for the folding of doxygen 
comments.
  
  I don't think we need to have an extra bool, one might later internally 
handle it differently with extra attributes in the hl files, like a 
lookAheadEndRegion as proposed, if we really don't like the general change.

REPOSITORY
  R216 Syntax Highlighting

BRANCH
  fix-region-lookahead

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

To: nibags, #framework_syntax_highlighting, cullmann, dhaumann
Cc: andreasgr, kwrite-devel, kde-frameworks-devel, hase, michaelh, ngraham, 
bruns, demsking, cullmann, sars, dhaumann

Reply via email to