[Lldb-commits] [lldb] [lldb] Provide default impl for MightHaveChildren (NFC) (PR #119977)

via lldb-commits lldb-commits at lists.llvm.org
Fri Feb 14 18:34:56 PST 2025


https://github.com/jimingham approved this pull request.

Ack, sorry I missed this...

This is fine.  You won't miss the fact that you should have overridden it to return false when you meant true, since then you'll get asked to make children but you can't.  So having a default implementation shouldn't cause mysterious failures because you didn't notice the function.

Good to remove all that boilerplate too!

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


More information about the lldb-commits mailing list