<div dir="auto"><div><br><div class="gmail_extra"><br><div class="gmail_quote">On Feb 23, 2017 8:48 PM, "Haojian Wu via Phabricator" <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="quoted-text">hokein added inline comments.<br>
<br>
<br>
================<br>
</div>Comment at: unittests/include-fixer/find-<wbr>all-symbols/<wbr>FindAllSymbolsTests.cpp:40<br>
+  void reportSymbols(llvm::StringRef FileName,<br>
+                     SymbolInfo::SignalMap NewSymbols) override {<br>
+    for (const auto &Entry : NewSymbols)<br>
----------------<br>
A new catch: `NewSymbols` should be passed by reference, otherwise a copy will be generated.<br></blockquote></div></div></div><div dir="auto">I did actually intend by-value here, FindAllSymbols no longer needs the map once it's reported, so it is moved rather than copied (see the end of FindAllSymbols.cpp)</div><div dir="auto"><br></div><div dir="auto">If this is too surprising, I can change it.</div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br><br>
<br>
<a href="https://reviews.llvm.org/D30210" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D30210</a><br>
<br>
<br>
<br>
</blockquote></div><br></div></div></div>