D16253: [Scheduler] Remove extraneous set of content indexing flag

Stefan BrĂ¼ns noreply at phabricator.kde.org
Tue Oct 16 18:23:29 BST 2018


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

REVISION SUMMARY
  The content indexing flag is already set by the BasicIndexingJob::index()
  in ModifiedFileIndexer::run() executed previously, no need to start
  another write transaction. Also, the flag is also set when the config
  only specifies BasicIndexing, i.e. content indexing was always done.

REPOSITORY
  R293 Baloo

BRANCH
  indexer_cleanup

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

AFFECTED FILES
  src/file/modifiedfileindexer.cpp

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


More information about the Kde-frameworks-devel mailing list