github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. 
:warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff HEAD~1 HEAD --extensions cpp -- 
clang-tools-extra/include-cleaner/test/tool.cpp 
clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp 
b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
index dd853950d..ba7c9f676 100644
--- a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
+++ b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
@@ -92,7 +92,8 @@ cl::opt<bool> Edit{
 };
 cl::opt<bool> Insert{
     "insert",
-    cl::desc("Allow header insertions (deprecated. Use -disable-insert 
instead)"),
+    cl::desc(
+        "Allow header insertions (deprecated. Use -disable-insert instead)"),
     cl::init(true),
     cl::cat(IncludeCleaner),
 };
@@ -196,17 +197,19 @@ private:
                 getCompilerInstance().getPreprocessor(), HeaderFilter);
 
     if (!Insert) {
-      llvm::errs() << "`-insert=0` is deprecated in favor of 
`-disable-insert`. "
-                      "The old flag was confusing since it suggested that 
inserts "
-                      "were disabled by default, when they were actually 
enabled. "
-                      "See 
https://github.com/llvm/llvm-project/issues/132983\n";;
+      llvm::errs()
+          << "`-insert=0` is deprecated in favor of `-disable-insert`. "
+             "The old flag was confusing since it suggested that inserts "
+             "were disabled by default, when they were actually enabled. "
+             "See https://github.com/llvm/llvm-project/issues/132983\n";;
     }
 
     if (!Remove) {
-      llvm::errs() << "`-remove=0` is deprecated in favor of 
`-disable-remove`. "
-                      "The old flag was confusing since it suggested that 
removes "
-                      "were disabled by default, when they were actually 
enabled. "
-                      "See 
https://github.com/llvm/llvm-project/issues/132983\n";;
+      llvm::errs()
+          << "`-remove=0` is deprecated in favor of `-disable-remove`. "
+             "The old flag was confusing since it suggested that removes "
+             "were disabled by default, when they were actually enabled. "
+             "See https://github.com/llvm/llvm-project/issues/132983\n";;
     }
 
     if (!Insert || DisableInsert)

``````````

</details>


https://github.com/llvm/llvm-project/pull/132991
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to