From: Tom Tromey Date: Sun, 22 Apr 2018 16:26:18 +0000 (-0600) Subject: Use std::string in mdebugread.c X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=795afcbbb4b6c9a47597b9da57221b1bf9fdc88f;p=binutils-gdb.git Use std::string in mdebugread.c This changes a couple of spots in mdebugread to use std::string rather than manual management. This is simpler, and also avoids shadowing by renaming the variable in question. gdb/ChangeLog 2018-10-04 Tom Tromey * mdebugread.c (parse_partial_symbols): Use std::string. --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 62840512ae3..0b5f35b2b7c 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,7 @@ +2018-10-04 Tom Tromey + + * mdebugread.c (parse_partial_symbols): Use std::string. + 2018-10-04 Tom Tromey * ctf.c (SET_ARRAY_FIELD): Rename "u32". diff --git a/gdb/mdebugread.c b/gdb/mdebugread.c index 453b8d5307d..62de80ee1e6 100644 --- a/gdb/mdebugread.c +++ b/gdb/mdebugread.c @@ -3197,13 +3197,9 @@ parse_partial_symbols (minimal_symbol_reader &reader, case 'f': if (! pst) { - int name_len = p - namestring; - char *name = (char *) xmalloc (name_len + 1); - - memcpy (name, namestring, name_len); - name[name_len] = '\0'; - function_outside_compilation_unit_complaint (name); - xfree (name); + std::string copy (namestring, p); + function_outside_compilation_unit_complaint + (copy.c_str ()); } add_psymbol_to_list (namestring, p - namestring, 1, VAR_DOMAIN, LOC_BLOCK, @@ -3220,13 +3216,9 @@ parse_partial_symbols (minimal_symbol_reader &reader, case 'F': if (! pst) { - int name_len = p - namestring; - char *name = (char *) xmalloc (name_len + 1); - - memcpy (name, namestring, name_len); - name[name_len] = '\0'; - function_outside_compilation_unit_complaint (name); - xfree (name); + std::string copy (namestring, p); + function_outside_compilation_unit_complaint + (copy.c_str ()); } add_psymbol_to_list (namestring, p - namestring, 1, VAR_DOMAIN, LOC_BLOCK,