chh abandoned this revision.
chh added a comment.
Included in https://reviews.llvm.org/D36761.
https://reviews.llvm.org/D36759
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Eugene.Zelenko added a comment.
Please abandon this and other similar revisions as obsoleted by
https://reviews.llvm.org/D36761.
https://reviews.llvm.org/D36759
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/m
chh added a comment.
Okay, 3 changes are combined into https://reviews.llvm.org/D36761.
https://reviews.llvm.org/D36759
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Eugene.Zelenko added a comment.
In https://reviews.llvm.org/D36759#842367, @chh wrote:
> Eugene, do you mean combining this one, https://reviews.llvm.org/D36756, and
> https://reviews.llvm.org/D36755 into one?
> I don't mind either way.
Sure. All three diffs are of same kind.
https://revie
chh added a comment.
Eugene, do you mean combining this one, https://reviews.llvm.org/D36756, and
https://reviews.llvm.org/D36755 into one?
I don't mind either way.
https://reviews.llvm.org/D36759
___
cfe-commits mailing list
cfe-commits@lists.llv
Eugene.Zelenko added a comment.
I think you could combine this refactoring into single change.
https://reviews.llvm.org/D36759
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
chh created this revision.
Simplify registerMatchers and check functions in CloexecFopenCheck.
https://reviews.llvm.org/D36759
Files:
clang-tidy/android/CloexecFopenCheck.cpp
clang-tidy/android/CloexecFopenCheck.h
Index: clang-tidy/android/CloexecFopenCheck.h
=
