[PATCH] D30210: [include-fixer] Add usage count to find-all-symbols.
Haojian Wu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Feb 24 02:54:40 PST 2017
hokein added a comment.
Thanks, still LGTM with one nit.
================
Comment at: include-fixer/find-all-symbols/FindAllSymbols.cpp:262
+ if (Filename != "") {
+ Reporter->reportSymbols(Filename, std::move(FileSymbols));
+ FileSymbols = {};
----------------
We don't need `std::move` right now.
https://reviews.llvm.org/D30210
More information about the cfe-commits
mailing list