[PATCH] D30210: [include-fixer] Add usage count to find-all-symbols.
Benjamin Kramer via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Feb 27 13:46:00 PST 2017
bkramer accepted this revision.
bkramer added a comment.
lg
================
Comment at: include-fixer/SymbolIndexManager.cpp:156
+ for (const auto &SymAndSig : MatchedSymbols)
+ Res.push_back(SymAndSig.Symbol);
+ return Res;
----------------
std::move
================
Comment at: include-fixer/find-all-symbols/FindAllMacros.cpp:67
+ FileSymbols);
+ FileSymbols = {};
}
----------------
Call me oldschool, but I still prefer `FileSymbols.clear()`
https://reviews.llvm.org/D30210
More information about the cfe-commits
mailing list