[PATCH] D52984: [analyzer] Checker reviewer's checklist

Umann Kristóf via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Oct 28 05:45:43 PDT 2018


Szelethus added a comment.

One more thing.

>From what I've seen, new checkers are in an overwhelming majority (again, from what **I've** seen) made by beginners, so I'd propose to include comments in checkers that are at least in part understandable by a beginner. I don't mean to make make 60% of the code grey, but there are a lot of unexplained non-trivial hackery in some.


https://reviews.llvm.org/D52984





More information about the cfe-commits mailing list