[Extractor] Fix progress reporting
ClosedPublic

Authored by bruns on Apr 18 2020, 6:46 PM.

Details

Summary

Commit 125b4db3230b ("Port endl to \n or std::cout/cerr as in qt5.15 endl
is Qt:: namespaced") removed the implicit "endl" flush. As a result,
the startedIndexing and finishedIndexing signals would be issued at the
same time.

This is especially bad when indexing large files, as both the KInfoCenter
qml component and balooctl monitor would report the previous file (or
no file at all, if only a single file is changed).

Test Plan

start kinfocenter / balooctl monitor
touch some huge PDF files

Diff Detail

Repository
R293 Baloo
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
bruns created this revision.Apr 18 2020, 6:46 PM
Restricted Application added projects: Frameworks, Baloo. · View Herald TranscriptApr 18 2020, 6:46 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
bruns requested review of this revision.Apr 18 2020, 6:46 PM
apol accepted this revision.Apr 19 2020, 3:08 PM
This revision is now accepted and ready to land.Apr 19 2020, 3:08 PM
This revision was automatically updated to reflect the committed changes.