https://github.com/Michael137 commented:
My original idea was that we can just call `CalculateNumChildren` and bubble up that error when something goes wrong. But if we're just going to consume it anyway, then keeping them as `CalculateNumChildrenIgnoringErrors` seems fine (though I would still prefer bubbling up errors, but that brings us back to the discussion about how we should be using `llvm::Expected`). CC @adrian-prantl @felipepiovezan https://github.com/llvm/llvm-project/pull/142607 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
