[Lldb-commits] [PATCH] D47302: [lldb, lldb-mi] Add method AddCurrentTargetSharedObjectPath to the SBDebugger.

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri May 25 01:19:21 PDT 2018

labath added a comment.

In https://reviews.llvm.org/D47302#1111641, @clayborg wrote:

> Be sure to not pass through any experimental settings.

About that, do we want to have some non-api breaking way of accessing the settings (like, `SBSomething GetSetting(std::string setting_name)`). This could be either an addition to the accessors for specific settings or even the only way of accessing the settings. In the former case, it would take the pressure off of adding a specific API for a setting we're not sure we want to keep. In the latter case, it would enable us to add/remove settings pretty much as we want without impacting api signature in anyway. Though, I guess it's arguable whether that would be doing anyone a favour as it would likely just replace a link error with a random runtime failure.

(I have no opinion whether we should do this or not, just throwing the idea out there.)



More information about the lldb-commits mailing list