aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSimon Marchi <simon.marchi@ericsson.com>2017-10-13 22:45:14 -0400
committerSimon Marchi <simon.marchi@ericsson.com>2017-10-13 22:45:14 -0400
commit3ec5942fbf8501db10784b0dff7d24222a113728 (patch)
tree58b5637e30b1680f8d888812c16f5897b3b661e9 /gdb/ada-typeprint.c
parentAutomatic date update in version.in (diff)
downloadbinutils-gdb-3ec5942fbf8501db10784b0dff7d24222a113728.tar.gz
binutils-gdb-3ec5942fbf8501db10784b0dff7d24222a113728.tar.bz2
binutils-gdb-3ec5942fbf8501db10784b0dff7d24222a113728.zip
ada: Use std::string in print_dynamic_range_bound
Replace this usage of GROW_VECT with an std::string. I don't think there's a reason for this variable to be static, other than it was cumbersome to manage its lifetime (i.e. use a cleanup) before. Tested by comparing the gdb.ada/*.exp test results before and after the patch. gdb/ChangeLog: * ada-typeprint.c (print_dynamic_range_bound): Use std::string.
Diffstat (limited to 'gdb/ada-typeprint.c')
-rw-r--r--gdb/ada-typeprint.c10
1 files changed, 3 insertions, 7 deletions
diff --git a/gdb/ada-typeprint.c b/gdb/ada-typeprint.c
index 2224bb3d79f..0d1f9dc117c 100644
--- a/gdb/ada-typeprint.c
+++ b/gdb/ada-typeprint.c
@@ -253,15 +253,11 @@ static void
print_dynamic_range_bound (struct type *type, const char *name, int name_len,
const char *suffix, struct ui_file *stream)
{
- static char *name_buf = NULL;
- static size_t name_buf_len = 0;
LONGEST B;
+ std::string name_buf (name, name_len);
+ name_buf += suffix;
- GROW_VECT (name_buf, name_buf_len, name_len + strlen (suffix) + 1);
- strncpy (name_buf, name, name_len);
- strcpy (name_buf + name_len, suffix);
-
- if (get_int_var_value (name_buf, B))
+ if (get_int_var_value (name_buf.c_str (), B))
ada_print_scalar (type, B, stream);
else
fprintf_filtered (stream, "?");