[Lldb-commits] [PATCH] D43837: Move option parsing out of the Args class

[Zachary Turner via Phabricator via lldb-commits]

zturner added inline comments.


================
Comment at: include/lldb/Interpreter/Options.h:123-126
+  llvm::Expected<Args> Parse(const Args &args,
+                             ExecutionContext *execution_context,
+                             lldb::PlatformSP platform_sp,
+                             bool require_validation);
----------------
It appears that all of these could be static functions.  Can we do that?


https://reviews.llvm.org/D43837