bruns added inline comments.
INLINE COMMENTS
> databasesanitizer.cpp:252
> + auto& summary = listResult.second;
> for (const auto& info: listResult.first) {
> + if (ignoredDevices[info.id] == false) {
If ignoredDevices is a Set/List, you can do a filter pass over the fileList
first.
auto& fileList = listResult.first;
auto tail = fileList.end();
for (auto deviceId : ignoredDevices) {
tail = std::remove_if(fileList.begin(), tail,
[deviceId] (const FileInfo& info) {
return info.id == deviceId;
});
}
summary.ignored += fileList.end() - tail;
std::erase(tail, fileList.end());
REPOSITORY
R293 Baloo
REVISION DETAIL
https://phabricator.kde.org/D12222
To: michaelh, #baloo, #frameworks
Cc: bruns, ashaposhnikov, michaelh, astippich, spoorun, ngraham
