[Lldb-commits] [PATCH] D100965: [lldb] Refactor argument group by SourceLocationSpec (NFCI)

[Med Ismail Bennani via Phabricator via lldb-commits]

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