[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