DavidSpickett added a reviewer: DavidSpickett. DavidSpickett added a comment.
Agreed, the original name sounds like a const method. Although I can see why it might have been this way because before optional there was no way to have a null return. So it's return a bool and a ref to the result. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146286/new/ https://reviews.llvm.org/D146286 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
