[Lldb-commits] [PATCH] Delete LLDB_DEFAULT_SHELL, and determine the default shell at runtime

[Greg Clayton]

You can't add members to any lldb::SB class, you will need to remove m_cached_shell and either put it into lldb_private::ProcessLaunchInfo. Why? Because we expect the lldb::SB API to be stable for people to link against and adding a new ivar changes the layout of the class. All lldb::SB classes must have a single member that is either a pointer, unique_ptr or shared_ptr and these can't change. Other rules for lldb::SB API classes include: no virtual functions and no inheritance.

http://reviews.llvm.org/D5805