================ @@ -806,6 +806,11 @@ def UseAfterLifetimeEnd : Checker<"UseAfterLifetimeEnd">, Dependencies<[LifetimeModeling]>, Documentation<NotDocumented>; +def ReportDanglingPtrDeref : Checker<"ReportDanglingPtrDeref">, ---------------- NagyDonat wrote:
> Let's address this in a followup commit. Renaming checkers is somewhat disruptive, even if it is just an alpha checker. I would prefer merging this with the right name if possible. > I think `DanglingPtrDerefChecker` or just `DanglingPtrDeref` aligns better > with the naming of existing checkers. The suffix `Checker` is also redundant, even more so than `Report`. Unfortunately it it is used by a few checkers, but we shouldn't introduce new checkers that use it. (By the way it is normal that the name of the _class that implements the checker_ is suffixed with `Checker`: e.g. `security.ArrayBound` is implemented in `class ArrayBoundChecker`.) https://github.com/llvm/llvm-project/pull/209278 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
