ioeric added a comment. lg!
================ Comment at: clangd/CodeComplete.cpp:1176 // This is available after Sema has run. - llvm::Optional<IncludeInserter> Inserter; // Available during runWithSema. + llvm::Optional<IncludeInserter> Inserter; // Optional during runWithSema. llvm::Optional<URIDistance> FileProximity; // Initialized once Sema runs. ---------------- Why optional? In the current implementation, it's always initialized. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60409/new/ https://reviews.llvm.org/D60409 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
