[PATCH] D33304: [clang-tidy] Add a new module Android and a new check for file descriptors.

Joerg Sonnenberger via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 18 09:42:37 PDT 2017

Previous message: [PATCH] D33304: [clang-tidy] Add a new module Android and a new check for file descriptors.
Next message: [PATCH] D33304: [clang-tidy] Add a new module Android and a new check for file descriptors.
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

joerg added a comment.

I find the use of "must" at the very least inappropriate. If there was no use case for not including it, it wouldn't be an option. There is also nothing really Android-specific here beside maybe the open64 mess.


Repository:
  rL LLVM

https://reviews.llvm.org/D33304

Previous message: [PATCH] D33304: [clang-tidy] Add a new module Android and a new check for file descriptors.
Next message: [PATCH] D33304: [clang-tidy] Add a new module Android and a new check for file descriptors.
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the cfe-commits mailing list