[Lldb-commits] [lldb] Change the return type of CalculateNumChildren to uint32_t. (PR #83501)

[Adrian Prantl via lldb-commits]

adrian-prantl wrote:

> Why not increase TypeSystem::GetNumChildren to return a size_t instead?

We could, but even 2^32 seems like an outrageous amount of children to me. What would be a use-case for this? A ValueObject that has the entire address space as synthetic children?

There is other code that assumes that we can store the result of GetNumChildren() in an int64 and use negative numbers as out-of-band signaling. We could replace that with std::optional of course.

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