[Lldb-commits] [lldb] [lldb] Fix MaxSummaryLength target property type (PR #72233)

[David Spickett via lldb-commits]

DavidSpickett wrote:

Thanks for the fix!

You'll be shocked (not shocked :) ) to know that there are zero tests for this setting currently. I'll have a look for an obvious place to put that, isn't difficult to test as you've shown.

I can't think of any reason to have negative summary lengths, so changing to unsigned doesn't seem like an issue.

https://github.com/llvm/llvm-project/pull/72233