mib marked an inline comment as done.
mib added inline comments.

================
Comment at: lldb/source/API/SBThread.cpp:853
+    if (!location_spec) {
+      sb_error.SetErrorString(
+          llvm::toString(location_spec.takeError()).c_str());
----------------
JDevlieghere wrote:
> Why not `sb_error = location_spec.takeError()`?
`takeError` returns an `llvm::Error` not a `lldb_private::Status`, and there is 
no `SBError` constructor for `Status`  so I don't think that's possible.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D100965/new/

https://reviews.llvm.org/D100965

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to