D9175: Migrate some more QRegExps to QRegularExpression

Christoph Cullmann noreply at phabricator.kde.org
Mon Feb 5 20:26:16 UTC 2018


cullmann added a comment.


  I think beside the
  
  > I think the old code tried to ensure we only add word-boundary if we still can find the string then. e.g. if the string that you highlight contains spaces or stuff like that at the borders, you don't add \b.
  
  all other comments are just further beautifications.
  
  Could that one thing be fixed and that commited?
  In any case the new stuff will be nicer than the old.

REPOSITORY
  R39 KTextEditor

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

To: dhaumann, cullmann, kfunk, mwolff
Cc: #frameworks, michaelh, kevinapavew, ngraham, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180205/d85a3cdd/attachment.html>


More information about the Kde-frameworks-devel mailing list