[Lldb-commits] [lldb] [lldb] Change the implementation of Status to store an llvm::Error (NFC) (PR #106774)

Jonas Devlieghere via lldb-commits lldb-commits at lists.llvm.org
Fri Aug 30 11:27:49 PDT 2024


================
@@ -91,10 +136,14 @@ class Status {
 
   ~Status();
 
-  // llvm::Error support
-  explicit Status(llvm::Error error) { *this = std::move(error); }
+  /// Try not to use this in new code. Migrate APIs to llvm::Expected instead.
----------------
JDevlieghere wrote:

[nit] I'd word this even stronger: "Avoid using this in new code.  Migrate APIs to use llvm::Error or llvm::Expected directly."

https://github.com/llvm/llvm-project/pull/106774


More information about the lldb-commits mailing list