[Lldb-commits] [PATCH] D48796: Refactoring for for the internal command line completion API (NFC)

[Jim Ingham via Phabricator via lldb-commits]

jingham added a comment.

This patch looks fine, I agree with Pavel the direction is good.  It seems like you would reduce a bunch of boiler-plate if you changed over InvokeCommonCompletionCallbacks to take a CompletionRequest.  Was there something that blocked you from doing that?


https://reviews.llvm.org/D48796