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

Stephen Hines via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 29 16:00:36 PDT 2019


srhines accepted this revision.
srhines added a comment.
This revision is now accepted and ready to land.

Everything looks great. Thanks for adding these improvements. While it's probably safe to commit this, perhaps you should give it 24 hours in case some of the other clang-tidy folks have different style or testing concerns.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62049





More information about the cfe-commits mailing list