=?utf-8?q?Donát?= Nagy <[email protected]>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/[email protected]>
================
@@ -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,
----------------
steakhal wrote:
In the other APIs, we pass BugTypes by reference. Is `BT` always non-null in
all callsites? If so, could we use reference here as well?
https://github.com/llvm/llvm-project/pull/156073
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
