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