ioeric added inline comments.

================
Comment at: lib/Tooling/Core/Replacement.cpp:578
+      continue;
+    ProcessedFileEntries.insert(FE);
+    Result[Entry.first] = std::move(Entry.second);
----------------
alexshap wrote:
> (saves 2 lines of code) (although not particularly important) 
>   if (ProcessedFileEntries.insert(FE).second)
>      Result[Entry.first] = std::move(Entry.second);
Always nice to save a few lines :) Thanks!


https://reviews.llvm.org/D26288



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

Reply via email to