ilya-biryukov added inline comments.
Comment at: include/clang/Sema/CodeCompleteConsumer.h:355
+/// \brief Get string representation of \p Kind, useful for for debugging.
+llvm::StringRef printCompletionKind(enum CodeCompletionContext::Kind Kind);
> nit: this doesn't print anything, maybe completionKindString?
This breaks LLVM [[
| style guide ]].
> Function names should be verb phrases.
And we every other function in this file seems to stick to these rules, I'd
rather be consistent with the rest of the code.
I'm leaning towards calling it `getCompletionKindAsString()`. It's not a short
name, but doesn't have the confusing `print` prefix. WDYT?
cfe-commits mailing list