Skip to content
Commit 064f4f2b authored by Stefan Brüns's avatar Stefan Brüns
Browse files

Split handling of on-demand extractor process from IndexedDataRetriever

Summary:
To allow a unification of the IndexedDataRetriever job and FileFetchJob,
move the process handling into a separate class and only keep
the UserMetaData handling.

As a small optimization, interleave the forking of the external process,
UserMetaData retrieval and joining of the external process. Also reuse
the QFileInfo in the constructor to determine if the file is writable and
thus has editable metadata.

Beyond the optimization, no user visible changes.

Depends on D25247

Test Plan:
When using a location not indexed by baloo, metadata is
still shown as before

Reviewers: #baloo, ngraham, astippich

Reviewed By: #baloo, ngraham, astippich

Tags: #baloo

Differential Revision: https://phabricator.kde.org/D25248
parent bb533b2d
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment