diff options
| author | Simon Marchi <simon.marchi@ericsson.com> | 2017-10-21 12:06:22 -0400 |
|---|---|---|
| committer | Simon Marchi <simon.marchi@polymtl.ca> | 2017-10-21 12:06:22 -0400 |
| commit | 6e17c56511104abd605bd2d122104467fc4f0089 (patch) | |
| tree | eb7995b7ea1d9f2348f698697f7a6d898374e05d | |
| parent | cfba98720f62473abc54457a376a3b7aa7ab1e40 (diff) | |
| download | gdb-6e17c56511104abd605bd2d122104467fc4f0089.zip gdb-6e17c56511104abd605bd2d122104467fc4f0089.tar.gz gdb-6e17c56511104abd605bd2d122104467fc4f0089.tar.bz2 | |
Use std::string in memory_map_parsing_data
Replace the fixed-size array with a string.
gdb/ChangeLog:
* memory-map.c (struct memory_map_parsing_data) <property_name>:
Change type to std::string.
(memory_map_start_property): Adjust.
(memory_map_end_property): Adjust.
| -rw-r--r-- | gdb/ChangeLog | 7 | ||||
| -rw-r--r-- | gdb/memory-map.c | 17 |
2 files changed, 16 insertions, 8 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index e440989..ecb73d9 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,10 @@ +2017-10-21 Simon Marchi <simon.marchi@ericsson.com> + + * memory-map.c (struct memory_map_parsing_data) <property_name>: + Change type to std::string. + (memory_map_start_property): Adjust. + (memory_map_end_property): Adjust. + 2017-10-21 Simon Marchi <simon.marchi@polymtl.ca> * infrun.h: Include common/byte-vector.h. diff --git a/gdb/memory-map.c b/gdb/memory-map.c index 0b21456..ff05394 100644 --- a/gdb/memory-map.c +++ b/gdb/memory-map.c @@ -43,10 +43,11 @@ parse_memory_map (const char *memory_map) /* Internal parsing data passed to all XML callbacks. */ struct memory_map_parsing_data - { - VEC(mem_region_s) **memory_map; - char property_name[32]; - }; +{ + VEC(mem_region_s) **memory_map; + + std::string property_name; +}; /* Handle the start of a <memory> element. */ @@ -103,7 +104,7 @@ memory_map_start_property (struct gdb_xml_parser *parser, char *name; name = (char *) xml_find_attribute (attributes, "name")->value; - snprintf (data->property_name, sizeof (data->property_name), "%s", name); + data->property_name.assign (name); } /* Handle the end of a <property> element and its value. */ @@ -115,16 +116,16 @@ memory_map_end_property (struct gdb_xml_parser *parser, { struct memory_map_parsing_data *data = (struct memory_map_parsing_data *) user_data; - char *name = data->property_name; - if (strcmp (name, "blocksize") == 0) + if (data->property_name == "blocksize") { struct mem_region *r = VEC_last (mem_region_s, *data->memory_map); r->attrib.blocksize = gdb_xml_parse_ulongest (parser, body_text); } else - gdb_xml_debug (parser, _("Unknown property \"%s\""), name); + gdb_xml_debug (parser, _("Unknown property \"%s\""), + data->property_name.c_str ()); } /* Discard the constructed memory map (if an error occurs). */ |