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

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to