[PATCH] D73520: [analyzer] BugReporterVisitors: Refactor and documentation

Csaba Dabis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 3 23:34:00 PST 2020


Charusso added inline comments.


================
Comment at: clang/include/clang/StaticAnalyzer/Core/BugReporter/BugReporterVisitors.h:44
 
-/// BugReporterVisitors are used to add custom diagnostics along a path.
+/// BugReporterVisitors are used to add custom diagnostics along a \emoji bug
+/// path. They also could serve false positive suppression.
----------------
Szelethus wrote:
> I suspect `\emoji` wasn't intentional here, given that it needs another argument :^)
Well, it is "So Pro" (- Apple) and I really like that. Someone wrote "along a path", but we are definitely mentioning the 🐛 path.
{F11475626}


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73520/new/

https://reviews.llvm.org/D73520





More information about the cfe-commits mailing list