[clang] [analyzer] Clean up bug types in CallAndMessageChecker (PR #156073)

[Donát Nagy via cfe-commits]

================
@@ -108,29 +109,26 @@ class CallAndMessageChecker
   bool PreVisitProcessArg(CheckerContext &C, SVal V, SourceRange ArgRange,
                           const Expr *ArgEx, int ArgumentNumber,
                           bool CheckUninitFields, const CallEvent &Call,
-                          std::unique_ptr<BugType> &BT,
+                          const BugType &BT,
                           const ParmVarDecl *ParamDecl) const;
 
-  static void emitBadCall(BugType *BT, CheckerContext &C, const Expr *BadE);
+  static void emitBadCall(const BugType *BT, CheckerContext &C,
----------------
NagyDonat wrote:

Nice catch! It's probably always non-null, but I'll double-check.

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