D21539: [FileIndexScheduler] Avoid emitting state changes multiple times

[Stefan Brüns]

bruns created this revision.
Herald added projects: Frameworks, Baloo.
Herald added subscribers: Baloo, kde-frameworks-devel.
bruns requested review of this revision.

REVISION SUMMARY
  FileContentIndexer::quit() will cause emission of the done() signal,
  triggering scheduleIndexing() later. scheduleIndexing() sets the
  current indexer state on its own and also emits the stateChanged signal,
  there is no need to do it from the various slots.

TEST PLAN
  unplug AC, state change signal is still emitted

REPOSITORY
  R293 Baloo

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

AFFECTED FILES
  src/file/fileindexscheduler.cpp

To: bruns
Cc: kde-frameworks-devel, #baloo, gennad, 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/20190602/8629e73e/attachment.html>