poboiko added a comment.
I like it, it's better than D15959: Wait for the extraction process to finish before scheduling <https://phabricator.kde.org/D15959>. And it seems to be working, as far as I can see :) INLINE COMMENTS > fileindexscheduler.h:57 > m_newFiles << file; > - QTimer::singleShot(0, this, > &FileIndexScheduler::scheduleIndexing); > + if (m_indexerState == Idle || m_isGoingIdle) { > + QTimer::singleShot(0, this, > &FileIndexScheduler::scheduleIndexing); But the very same check is already in the first line of `scheduleIndexing`, why do we need it also here? REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D16265 To: bruns, #baloo, #frameworks, poboiko, ngraham Cc: kde-frameworks-devel, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams