[Lldb-commits] [PATCH] D30789: Make file and directory name completion work properly on Windows.
Zachary Turner via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Mar 9 13:44:57 PST 2017
zturner added a comment.
In https://reviews.llvm.org/D30789#696870, @jingham wrote:
> What is the motivation behind of the "2" versions of the functions you added?
Ahh yea. I can call them something else if you have a better name. But basically in a unit test I don't have an instance of a `CommandInterpreter` object, and creating one is non-trivial (it requires a `Debugger` instance, which requires calling `Debugger::CreateInstance()`, which depends on a lot of other system initialization having been done.
But none of the methods I was trying to test use the `CommandInterpreter` anyway, so I just needed some kind of interface into the completion that didn't require me to pass one in.
More information about the lldb-commits