ilya-biryukov added a comment. Only one important comment about bringing back the comment.
================ Comment at: clangd/index/Background.cpp:224 + auto NeedsReIndexing = loadShards(std::move(ChangedFiles)); + // Run indexing for files that needs to be updated. + std::shuffle(NeedsReIndexing.begin(), NeedsReIndexing.end(), ---------------- a typo: s/that needs/that need/ ================ Comment at: clangd/index/Background.cpp:344 + DigestIt.first->second = Hash; + IndexedSymbols.update(Path, std::move(SS), std::move(RS)); + } ---------------- Let's bring back the comment about rewriting the newer version. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55224/new/ https://reviews.llvm.org/D55224 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
