[Lldb-commits] [lldb] [lldb] Deprecate SBBreakpoint::AddName in favor of AddNameWithErrorHandling (PR #71228)

[Alex Langford via lldb-commits]

https://github.com/bulbazord created https://github.com/llvm/llvm-project/pull/71228

AddName gives no feedback other than if it succeeded whereas AddNameWithErrorHandling gives you back an SBError object. I would like to mark AddName as deprecated and direct folks to use AddNameWithErorrHandling instead.

>From e1e99bdf51d559aa9c96cfd733f12b35e8db7728 Mon Sep 17 00:00:00 2001
From: Alex Langford <alangford at apple.com>
Date: Fri, 3 Nov 2023 13:24:27 -0700
Subject: [PATCH] [lldb] Deprecate SBBreakpoint::AddName in favor of
 AddNameWithErrorHandling

AddName gives no feedback other than if it succeeded whereas
AddNameWithErrorHandling gives you back an SBError object.
I would like to mark AddName as deprecated and direct folks to use
AddNameWithErorrHandling instead.
---
 lldb/include/lldb/API/SBBreakpoint.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lldb/include/lldb/API/SBBreakpoint.h b/lldb/include/lldb/API/SBBreakpoint.h
index 0bb7c31d74f21f9..c7ef287a5a6f17b 100644
--- a/lldb/include/lldb/API/SBBreakpoint.h
+++ b/lldb/include/lldb/API/SBBreakpoint.h
@@ -112,6 +112,7 @@ class LLDB_API SBBreakpoint {
 
   SBError SetScriptCallbackBody(const char *script_body_text);
 
+  LLDB_DEPRECATED("Use AddNameWithErrorHandling instead")
   bool AddName(const char *new_name);
 
   SBError AddNameWithErrorHandling(const char *new_name);