[PATCH] D33304: [clang-tidy][Part1] Add a new module Android and three new checks.

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 2 03:46:45 PDT 2017


hokein added a comment.

Thanks for the contributions.

All your three checks are not android specific -- because they are checking POSIX APIs (`open`, `creat`, `fopen`), which are more likely related to POSIX. So personally, I'm +1 on a "posix" module, instead of "android", but wait to see other reviewers' opinions before renaming it.


https://reviews.llvm.org/D33304





More information about the cfe-commits mailing list