kadircet accepted this revision. kadircet added a comment. This revision is now accepted and ready to land.
thanks! ================ Comment at: clang-tools-extra/clangd/IncludeCleaner.cpp:127 + // declared multiple times. The most common two cases are: + // - Definition available in TU, only mark that one as usage. rest is + // likely to be unnecessary. this might result in false positives when an ---------------- i know it was me who didn't capitalize, but can you capitalize the first words :D s/rest/Rest s/this/This (line below) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114949/new/ https://reviews.llvm.org/D114949 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
