[RFA,02/42] Change buildsym_compunit::comp_dir to be a unique_xmalloc_ptr
Commit Message
This change buildsym_compunit::comp_dir to be a unique_xmalloc_ptr.
This is just a small cleanup to remove some manual memory management.
gdb/ChangeLog
2018-05-22 Tom Tromey <tom@tromey.com>
* buildsym.c (~buildsym_compunit): Update.
(struct buildsym_compunit) <comp_unit>: Now a unique_xmalloc_ptr.
(start_subfile, patch_subfile_names)
(end_symtab_with_blockvector): Update.
---
gdb/ChangeLog | 7 +++++++
gdb/buildsym.c | 11 +++++------
2 files changed, 12 insertions(+), 6 deletions(-)
Comments
On 2018-05-23 12:58 AM, Tom Tromey wrote:
> This change buildsym_compunit::comp_dir to be a unique_xmalloc_ptr.
> This is just a small cleanup to remove some manual memory management.
LGTM.
Simon
@@ -126,7 +126,6 @@ struct buildsym_compunit
xfree (subfile->line_vector);
xfree (subfile);
}
- xfree (comp_dir);
}
/* The objfile we're reading debug info from. */
@@ -142,7 +141,7 @@ struct buildsym_compunit
struct subfile *main_subfile = nullptr;
/* E.g., DW_AT_comp_dir if DWARF. Space for this is malloc'd. */
- char *comp_dir;
+ gdb::unique_xmalloc_ptr<char> comp_dir;
/* Space for this is not malloc'd, and is assumed to have at least
the same lifetime as objfile. */
@@ -703,7 +702,7 @@ start_subfile (const char *name)
gdb_assert (buildsym_compunit != NULL);
- subfile_dirname = buildsym_compunit->comp_dir;
+ subfile_dirname = buildsym_compunit->comp_dir.get ();
/* See if this subfile is already registered. */
@@ -824,7 +823,7 @@ patch_subfile_names (struct subfile *subfile, const char *name)
&& subfile->name != NULL
&& IS_DIR_SEPARATOR (subfile->name[strlen (subfile->name) - 1]))
{
- buildsym_compunit->comp_dir = subfile->name;
+ buildsym_compunit->comp_dir.reset (subfile->name);
subfile->name = xstrdup (name);
set_last_source_file (name);
@@ -1410,10 +1409,10 @@ end_symtab_with_blockvector (struct block *static_block,
if (buildsym_compunit->comp_dir != NULL)
{
/* Reallocate the dirname on the symbol obstack. */
+ const char *comp_dir = buildsym_compunit->comp_dir.get ();
COMPUNIT_DIRNAME (cu)
= (const char *) obstack_copy0 (&objfile->objfile_obstack,
- buildsym_compunit->comp_dir,
- strlen (buildsym_compunit->comp_dir));
+ comp_dir, strlen (comp_dir));
}
/* Save the debug format string (if any) in the symtab. */