[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