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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
