D21583: [FilteredDirIterator] Combine all suffixes into one large RegExp

Stefan Brüns noreply at phabricator.kde.org
Tue Jun 4 19:08:12 BST 2019


bruns created this revision.
bruns added reviewers: Baloo, ngraham, poboiko.
Herald added projects: Frameworks, Baloo.
Herald added a subscriber: kde-frameworks-devel.
bruns requested review of this revision.

REVISION SUMMARY
  Most of the about 50 default filters are suffix matches, which can be
  easily combined into one large regular expression. Doing so reduces
  the matching cost by about 80%.

TEST PLAN
  valgrind --tool=callgrind baloo_file

REPOSITORY
  R293 Baloo

BRANCH
  submit_unindexed

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

AFFECTED FILES
  src/file/regexpcache.cpp

To: bruns, #baloo, ngraham, poboiko
Cc: kde-frameworks-devel, LeGast00n, domson, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190604/abf4a6f0/attachment.html>


More information about the Kde-frameworks-devel mailing list