[Lldb-commits] [PATCH] D44306: Move Args::StringToAddress to Target::EvaluateAddressExpression

Jim Ingham via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Mar 14 11:34:13 PDT 2018

jingham added a comment.

Except for the to -> To to keep consistent with all the other lldb function naming this looks fine.

Now that they are all together it's easy to see we haven't been consistent in these functions.  We really should make ToFormat return the format & take an error reference, and have ToBoolean take an error so callers don't have to cons it up.  But that's orthogonal to this patch.

Comment at: include/lldb/Interpreter/OptionArgParser.h:18-19
+struct OptionArgParser {
+  static lldb::addr_t toAddress(const ExecutionContext *exe_ctx,
+                                llvm::StringRef s, lldb::addr_t fail_value,
+                                Status *error);
Should be ToAddress.


More information about the lldb-commits mailing list