D23457: Port regex search to QRegularExpression

Ahmad Samir noreply at phabricator.kde.org
Sun Aug 25 23:47:17 BST 2019


ahmadsamir added a comment.


  Maybe they'll also see it as ktexteditor/kate using a regex engine that matches what the abundance of online pcre docs say, and how other editors that use pcre behave?
  
  IIUC, '\s' was workedaround so as not to match a newline so that the search pattern wouldn't be considered multiline (isMultiLine() function), which makes findAll and replaceAll slower as it took longer, v.s. just matching against each line separately.
  
  The thing is, what kateregexp did was replace '\s' with '[ \t]', which users who want this behaviour can easily use.
  
  Technically it's a whole new class, QRegularExpression, some different behaviours are sort of expected...

REPOSITORY
  R39 KTextEditor

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

To: ahmadsamir, #ktexteditor, dhaumann, cullmann
Cc: kde-frameworks-devel, kwrite-devel, LeGast00n, GB_2, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20190825/0cdf0f8e/attachment.html>


More information about the KWrite-Devel mailing list