[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