[Lldb-commits] [PATCH] D66453: Make the FindTypes(std::vector<CompilerContext>, ...) API testable in lldb-test
Jonas Devlieghere via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Aug 19 17:48:16 PDT 2019
JDevlieghere added inline comments.
Comment at: lldb/tools/lldb-test/lldb-test.cpp:145
+ cl::desc("Specify a compiler context as \"[(type,name),...]\"."),
+ cl::value_desc("context"), cl::sub(SymbolsSubcommand));
> JDevlieghere wrote:
> > > ...as a JSON array...`
> > This makes me wonder though, wouldn't it be better to pass a file here and have it live in inputs? Do you expect it to be different across tests or could we reuse the context across tests?
> I don't really expect a DeclContexts to have more than 3 entries in the forseeable future, so that would arguable just make the test harder to read.
But would those entries be the same across tests or not? If they're the same I think it'd be better to reuse them, even if it's only a few entries.
CHANGES SINCE LAST ACTION
More information about the lldb-commits