[PATCH] D62412: [clang-tidy] Fix unused-variable warning after r361647.

2019-05-27 Thread Haojian Wu via Phabricator via cfe-commits
hokein added a comment. I have commit this patch on behalf of @ymandel, as this warning blocks our internal integration. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62412/new/ https://reviews.llvm.org/D62412 ___

[PATCH] D62412: [clang-tidy] Fix unused-variable warning after r361647.

2019-05-27 Thread Haojian Wu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rCTE361749: [clang-tidy] Fix unused-variable warning after r361647. (authored by hokein, committed by ). Changed prior to commit: https://reviews.llvm.org/D62412?vs=201289&id=201481#toc Repository: rCT

[PATCH] D62412: [clang-tidy] Fix unused-variable warning after r361647.

2019-05-27 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov accepted this revision. ilya-biryukov added a comment. This revision is now accepted and ready to land. LGTM Comment at: clang-tools-extra/clang-tidy/utils/TransformerClangTidyCheck.cpp:21 : ClangTidyCheck(Name, Context), Rule(std::move(R)) { - for (const au