D27766: [ModifiedFileIndexer] Omit BasicIndexingJob run when not required

[Anthony Fieroni]

anthonyfieroni added inline comments.

INLINE COMMENTS

> modifiedfileindexer.cpp:118
>          if (tr.hasDocument(job.document().id())) {
>              if (cTimeChanged) {
>                  tr.replaceDocument(job.document(), XAttrTerms | DocumentTime | FileNameTerms | DocumentUrl);

This check is redundant now, we here only when `cTimeChanged` is true.

REPOSITORY
  R293 Baloo

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

To: bruns, #baloo, ngraham
Cc: anthonyfieroni, kde-frameworks-devel, hurikhan77, lots0logs, LeGast00n, cblack, fbampaloukas, GB_2, 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/20200301/a2e2e176/attachment.html>