[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