[PATCH] D61967: [clang-tidy] Add a close-on-exec check on pipe() in Android module.

Dmitri Gribenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 4 01:41:19 PDT 2019


gribozavr accepted this revision.
gribozavr added inline comments.


================
Comment at: clang-tools-extra/test/clang-tidy/android-cloexec-pipe.cpp:14
+int pipe(int pipefd[2]);
+void noWarning() {
+  int pipefd[2];
----------------
noWarningForPipeInNamespace ?


================
Comment at: clang-tools-extra/test/clang-tidy/android-cloexec-pipe.cpp:23
+  int pipe(int pipefd[2]);
+  void noWarning() {
+    int pipefd[2];
----------------
noWarningForPipeMemberFunction ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61967





More information about the cfe-commits mailing list