[clang] [analyzer] Prettify checker registration and unittest code (PR #147797)

Balazs Benics via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 10 06:34:49 PDT 2025


=?utf-8?q?Donát?= Nagy <donat.nagy at ericsson.com>,
=?utf-8?q?Donát?= Nagy <donat.nagy at ericsson.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/147797 at github.com>


================
@@ -112,26 +112,35 @@ class CheckerRegistry {
     return true;
   }
 
-public:
-  /// Adds a checker to the registry. Use this non-templated overload when your
-  /// checker requires custom initialization.
-  void addChecker(RegisterCheckerFn Fn, ShouldRegisterFunction sfn,
+  /// Adds a checker to the registry. This private, most general variant is
----------------
steakhal wrote:

```suggestion
  /// Adds a checker to the registry.
  /// This private, most general overload is...
```

Could you make the `Adds a checker to the registry.` the first and dedicated line of all of the overloads?
That would make these stand out thus easier to read.

https://github.com/llvm/llvm-project/pull/147797


More information about the cfe-commits mailing list