[PATCH] D41537: Optionally add code completion results for arrow instead of dot
Ivan Donchevskii via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri May 25 00:27:56 PDT 2018
yvvan marked an inline comment as done.
yvvan added inline comments.
Comment at: lib/Sema/SemaCodeComplete.cpp:4148
+ CompletionSucceded |= DoCompletion(Base, IsArrow, None);
> ilya-biryukov wrote:
> > NIT: maybe swap the two cases to do the non-fixit ones first? It is the most common case, so it should probably go first.
> The actual completion should be the last because of the "Completion context". I've put them in this order intentionally. I will add comment about that.
I've rechecked this part - I can actually now move it without breaking anything
More information about the cfe-commits