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

Jim Ingham via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Mar 12 12:16:36 PDT 2018


jingham added a comment.

I liked the fact that all these Argument input type string converters were all in one place, though arguably Args was not the right place.  Among other things it made it easy to see that if you were adding a new command that took in an Address, AddressToString was the canonical way to do it.  That's less obvious now that this one is a target function, and you would have to go look at another argument to see what it does.  I wonder if this problem isn't better solved by moving all the parsing functions somewhere more appropriate (like CommandInterpreter?) and then it isn't surprising what they might do.


https://reviews.llvm.org/D44306





More information about the lldb-commits mailing list