================
@@ -353,7 +356,8 @@ ClangTidyASTConsumerFactory::ClangTidyASTConsumerFactory(
   if (Context.canExperimentalCustomChecks() && custom::RegisterCustomChecks)
     custom::RegisterCustomChecks(Context.getOptions(), *CheckFactories);
 #endif
-  for (ClangTidyModuleRegistry::entry E : ClangTidyModuleRegistry::entries()) {
+  for (ClangTidyModuleRegistry::entry const E :
----------------
localspook wrote:

```suggestion
  for (const ClangTidyModuleRegistry::entry E :
```

But shouldn't clang-format be able to police const placement? 
https://clang.llvm.org/docs/ClangFormatStyleOptions.html#qualifierorder

https://github.com/llvm/llvm-project/pull/167030
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to