bruns created this revision. bruns added reviewers: Baloo, ngraham. Herald added projects: Frameworks, Baloo. Herald added a subscriber: kde-frameworks-devel. bruns requested review of this revision.
REVISION SUMMARY In case the index level is set to "non content indexing", the file was still flagged when it had changed. As the content indexer checks for the level itself (the config may have changed in the meantime), the effect is limited, but the code is still wrong. In case the level is set to "content indexing", the flag would be set twice. As it is a flag the effect is limited, but it blows up the transaction unnecessarily. REPOSITORY R293 Baloo BRANCH submit REVISION DETAIL https://phabricator.kde.org/D28326 AFFECTED FILES src/file/unindexedfileindexer.cpp To: bruns, #baloo, ngraham Cc: kde-frameworks-devel, hurikhan77, lots0logs, LeGast00n, cblack, fbampaloukas, GB_2, domson, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams