[clang-tools-extra] [clangd] Dont require confirmation for include-cleaner batch-fixes (PR #76826)
Haojian Wu via cfe-commits
cfe-commits at lists.llvm.org
Thu Jan 4 01:00:13 PST 2024
================
@@ -237,18 +236,6 @@ removeAllUnusedIncludes(llvm::ArrayRef<Diag> UnusedIncludes) {
Diag.Fixes.front().Edits.begin(),
Diag.Fixes.front().Edits.end());
}
-
- // TODO(hokein): emit a suitable text for the label.
- ChangeAnnotation Annotation = {/*label=*/"",
----------------
hokein wrote:
(I can't comment on the unchanged code), as we don't use ChangeAnnotation for both cases, I think we can remove the annotation-propagation code (line 271-274) in `fixAll`.
https://github.com/llvm/llvm-project/pull/76826
More information about the cfe-commits
mailing list