[Lldb-commits] [lldb] [LLDB] Make the thread list for SBSaveCoreOptions iterable (PR #122541)

Fri, 10 Jan 2025 15:01:48 -0800 

================
@@ -47,6 +46,9 @@ class SaveCoreOptions {
 
   void AddMemoryRegionToSave(const lldb_private::MemoryRegionInfo &region);
 
+  std::optional<lldb::ThreadSP> GetThreadAtIndex(uint32_t idx) const;
----------------
clayborg wrote:

We don't need optional if `lldb::ThreadSP` is what is returned, just return a 
`lldb::ThreadSP` and that can be checked for being invalid. There is no case 
where we want a valid optional with an empty `lldb::ThreadSP`.

https://github.com/llvm/llvm-project/pull/122541
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to