[Lldb-commits] [PATCH] D142926: [lldb] Replace SB swig interfaces with API headers

[Greg Clayton via Phabricator via lldb-commits]

clayborg added a comment.

In D142926#4111717 <https://reviews.llvm.org/D142926#4111717>, @bulbazord wrote:

> Potentially interesting:
>
> - SBData::GetDescription base_addr parameter has default value now
> - SBInstructionList::GetInstructionsCount canSetBreakpoint has default value now
> - SBMemoryRegionInfo::SBMemoryRegionInfo 3rd constructor parameter stack_memory has default value now

Make sure it does work. I seem to remember people sometimes making two functions in the .i file for default parameters. Can't remember if SWIG does the right thing with default params or not, but please test to ensure both work (with and without).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142926/new/

https://reviews.llvm.org/D142926