Will be used in clangd. See D43377.
Details
Diff Detail
- Repository
- rC Clang
- Build Status
Buildable 15061 Build 15061: arc lint + arc unit
Event Timeline
include/clang/Sema/CodeCompleteConsumer.h | ||
---|---|---|
355 | nit: this doesn't print anything, maybe completionKindString? |
include/clang/Sema/CodeCompleteConsumer.h | ||
---|---|---|
355 | This breaks LLVM style guide.
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? |
include/clang/Sema/CodeCompleteConsumer.h | ||
---|---|---|
355 | print or get is fine (you could drop the as). Not sure how I feel about this rule :-) |
nit: this doesn't print anything, maybe completionKindString?