bruns added a comment.

  good catch, but the fix is overengineered ...

INLINE COMMENTS

> monitor.cpp:90
> +    batchSize.waitForFinished();
>  
> +    if (!batchSize.isError()) {

much more trivial:

  auto updateInterval = std::max(100, 5 * m_scheduler->getBatchSize());
  if (m_filesIndexed % updateInterval == 0) { ...

REPOSITORY
  R293 Baloo

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

To: narvaez, #baloo, bruns, astippich, poboiko
Cc: kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, spoorun, 
ngraham, bruns, abrahams

Reply via email to