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

Alexander Polyakov via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu May 24 10:13:34 PDT 2018

polyakov.alex added a comment.

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

> It might make sense to create a new SBTargetSettings class that has accessors. Then we can have to accessors on SBTarget:
>   class SBTarget {
>     static SBTargetSettings GetGlobalSettings();
>     SBTargetSettings GetSettings();
>   };

What global settings should be in your opinion? I guess that they should be stored in the `SBDebugger`, and a typical use case for them would be:

  SBTarget target;

> This allows us to expose settings in a way that would allow us to serialize the settings and then load them again later.
>   class SBTargetSettings {
>     // Accessors for "target...." setting
>     void AppendImageSearchPath(const char *from, const char *to);
>     size_t GetNumImageSearchPaths();
>     const char *GetImageSearchPathAtIndex(size_t i);
>     // Save and load all settings
>     void Load(SBStream &s);
>     void Save(SBStream &s);
>   };

Serialization sounds good, but, to accurately understand you, do you mean "classic" serialization with saving data into the file or serialization just for the time when debugger is run?



More information about the lldb-commits mailing list