[PATCH] D30210: [include-fixer] Add usage count to find-all-symbols.

[Haojian Wu via Phabricator via cfe-commits]

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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits