* [PATCH] Use std::string in main_info
@ 2023-02-10 0:42 Tom Tromey
2023-02-10 16:53 ` John Baldwin
0 siblings, 1 reply; 2+ messages in thread
From: Tom Tromey @ 2023-02-10 0:42 UTC (permalink / raw)
To: gdb-patches; +Cc: Tom Tromey
This changes main_info to use std::string. It removes some manual
memory management.
---
gdb/symtab.c | 22 +++++++---------------
1 file changed, 7 insertions(+), 15 deletions(-)
diff --git a/gdb/symtab.c b/gdb/symtab.c
index 9d6ee388ca2..bd73c521b1a 100644
--- a/gdb/symtab.c
+++ b/gdb/symtab.c
@@ -104,16 +104,9 @@ static struct block_symbol
struct main_info
{
- main_info () = default;
-
- ~main_info ()
- {
- xfree (name_of_main);
- }
-
/* Name of "main". */
- char *name_of_main = nullptr;
+ std::string name_of_main;
/* Language of "main". */
@@ -6172,15 +6165,14 @@ set_main_name (const char *name, enum language lang)
{
struct main_info *info = get_main_info ();
- if (info->name_of_main != NULL)
+ if (!info->name_of_main.empty ())
{
- xfree (info->name_of_main);
- info->name_of_main = NULL;
+ info->name_of_main.clear ();
info->language_of_main = language_unknown;
}
if (name != NULL)
{
- info->name_of_main = xstrdup (name);
+ info->name_of_main = name;
info->language_of_main = lang;
}
}
@@ -6287,10 +6279,10 @@ main_name ()
{
struct main_info *info = get_main_info ();
- if (info->name_of_main == NULL)
+ if (info->name_of_main.empty ())
find_main_name ();
- return info->name_of_main;
+ return info->name_of_main.c_str ();
}
/* Return the language of the main function. If it is not known,
@@ -6301,7 +6293,7 @@ main_language (void)
{
struct main_info *info = get_main_info ();
- if (info->name_of_main == NULL)
+ if (info->name_of_main.empty ())
find_main_name ();
return info->language_of_main;
--
2.39.1
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] Use std::string in main_info
2023-02-10 0:42 [PATCH] Use std::string in main_info Tom Tromey
@ 2023-02-10 16:53 ` John Baldwin
0 siblings, 0 replies; 2+ messages in thread
From: John Baldwin @ 2023-02-10 16:53 UTC (permalink / raw)
To: Tom Tromey, gdb-patches
On 2/9/23 4:42 PM, Tom Tromey wrote:
> This changes main_info to use std::string. It removes some manual
> memory management.
LGTM
--
John Baldwin
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-02-10 16:53 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-10 0:42 [PATCH] Use std::string in main_info Tom Tromey
2023-02-10 16:53 ` John Baldwin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).