D15081: replace own wildcard matcher with QRegularExpression combining all wildcards add validator to index tool, only ? and * wildcard stuff is valid fix some syntax files that had e.g. , instead of ; and spaces mixed in remove asterisk definition, that...

Dominik Haumann noreply at phabricator.kde.org
Sat Aug 25 16:44:20 BST 2018


dhaumann added a comment.


  The old wildcard matcher existed for performance reasons. Did you check?
  
  PS: When doing review requests, could you please have a nice subject, and not put very long lines into its stead? :-p Especially since this line will also appear in Davids release notes?

REPOSITORY
  R216 Syntax Highlighting

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

To: cullmann, vkrause, dhaumann
Cc: kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, bruns, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180825/f9dc9bac/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list