[Lldb-commits] [PATCH] Add SBArgs to the public interface

[Greg Clayton]

I don't want the API if we aren't going to use it in our API.

If we do find a use for it we will need to returns "const char *" objects and state that the "const char *" values that are returned are only valid while for the lifetime of the SBArgs object. Then all std::string values turn into "const char *" in your API. Also, if you find a use for SBArgs somewhere you can't remove or change any existing API, you can only add functions. SBArgs could be returned from SBLaunchInfo, and the arguments could be set using an SBArgs (again, don't remove or change anything).

The other reason I would rather not expose this is it opens up all sorts of question and possibilities of wether people are going to expect arguments to be expanded just like a shell? Which shell will it emulate? And all these questions we have been mulling over internally in LLDB.


http://reviews.llvm.org/D8265

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/