[PATCH] Basic reform of chunks for overload completion results.
Francisco Lopes da Silva
francisco.mailing.lists at oblita.com
Wed Jan 28 04:50:05 PST 2015
================
Comment at: lib/Sema/SemaCodeComplete.cpp:2840-2841
@@ +2839,4 @@
+ Opt.AddChunk(CodeCompletionString::CK_Comma);
+ AddOverloadParameterChunks(Context, Policy, Function, Prototype, Opt,
+ CurrentArg, P, /*InOptional=*/true);
+ Result.AddOptionalChunk(Opt.TakeString());
----------------
klimek wrote:
> So we're both recursing and looping? This seems like a strange pattern to me, that at least needs a comment.
OK.
http://reviews.llvm.org/D7152
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the cfe-commits
mailing list