D28326: [UnindexedFileIndexer] Only mark file for content indexing when needed

[Stefan Brüns]

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

REVISION SUMMARY
  In case the index level is set to "non content indexing", the file was
  still flagged when it had changed. As the content indexer checks for
  the level itself (the config may have changed in the meantime), the
  effect is limited, but the code is still wrong.
  
  In case the level is set to "content indexing", the flag would be set
  twice. As it is a flag the effect is limited, but it blows up the
  transaction unnecessarily.

REPOSITORY
  R293 Baloo

BRANCH
  submit

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

AFFECTED FILES
  src/file/unindexedfileindexer.cpp

To: bruns, #baloo, ngraham
Cc: kde-frameworks-devel, hurikhan77, lots0logs, LeGast00n, cblack, fbampaloukas, GB_2, 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/20200326/6cdfeb1a/attachment-0001.html>