hokein added inline comments.
================ Comment at: clangd/index/dex/Dex.h:61 } // Symbols are owned by BackingData, Index takes ownership. + template <typename SymbolRange, typename RefsRange, typename Payload> ---------------- nit: this comment is stale too. ================ Comment at: unittests/clangd/DexTests.cpp:618 + Files.push_back(R.Location.FileURI); + }); +} ---------------- The test seems doesn't verify any thing? Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D52726 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
