ilya-biryukov added a comment.

And another NIT :-)



================
Comment at: lib/Sema/CodeCompleteConsumer.cpp:548
     OS << "COMPLETION: ";
+    std::vector<std::string> Tags;
     switch (Results[I].Kind) {
----------------
NIT: maybe move Tags into the corresponding case handler?

Would require putting a set of braces around it, but that shouldn't make the 
code less readable.


Repository:
  rC Clang

https://reviews.llvm.org/D53635



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

Reply via email to